Skip to content

Commit fc309ca

Browse files
committed
Implement Object.getOwnPropertyDescriptor function
JerryScript-DCO-1.0-Signed-off-by: Kristof Kosztyo [email protected]
1 parent d6c9c59 commit fc309ca

File tree

4 files changed

+141
-2
lines changed

4 files changed

+141
-2
lines changed

jerry-core/ecma/base/ecma-helpers.cpp

Lines changed: 29 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1239,6 +1239,35 @@ ecma_free_property_descriptor (ecma_property_descriptor_t *prop_desc_p) /**< pro
12391239
*prop_desc_p = ecma_make_empty_property_descriptor ();
12401240
} /* ecma_free_property_descriptor */
12411241

1242+
/**
1243+
* Return property descriptor of property.
1244+
*/
1245+
ecma_property_descriptor_t
1246+
ecma_get_property_descriptor_from_property (ecma_property_t *prop_p) /**< property */
1247+
{
1248+
ecma_property_descriptor_t prop_desc = ecma_make_empty_property_descriptor ();
1249+
1250+
prop_desc.is_enumerable = ecma_is_property_enumerable (prop_p);
1251+
prop_desc.is_configurable = ecma_is_property_configurable (prop_p);
1252+
1253+
if (prop_p->type == ECMA_PROPERTY_NAMEDDATA)
1254+
{
1255+
prop_desc.value = ecma_get_named_data_property_value (prop_p);
1256+
prop_desc.is_value_defined = true;
1257+
prop_desc.is_writable = ecma_is_property_writable (prop_p);
1258+
prop_desc.is_writable_defined = true;
1259+
}
1260+
else if (prop_p->type == ECMA_PROPERTY_NAMEDACCESSOR)
1261+
{
1262+
prop_desc.get_p = ecma_get_named_accessor_property_getter (prop_p);
1263+
prop_desc.is_get_defined = prop_desc.get_p != NULL ? true : false;
1264+
prop_desc.set_p = ecma_get_named_accessor_property_setter (prop_p);
1265+
prop_desc.is_set_defined = prop_desc.set_p != NULL ? true : false;
1266+
}
1267+
1268+
return prop_desc;
1269+
} /* ecma_get_property_descriptor_from_property */
1270+
12421271
/**
12431272
* @}
12441273
* @}

jerry-core/ecma/base/ecma-helpers.h

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -290,6 +290,7 @@ extern void ecma_set_property_lcached (ecma_property_t *prop_p,
290290

291291
extern ecma_property_descriptor_t ecma_make_empty_property_descriptor (void);
292292
extern void ecma_free_property_descriptor (ecma_property_descriptor_t *prop_desc_p);
293+
extern ecma_property_descriptor_t ecma_get_property_descriptor_from_property (ecma_property_t *prop_p);
293294

294295
/* ecma-helpers-external-pointers.c */
295296
extern bool

jerry-core/ecma/builtin-objects/ecma-builtin-object.cpp

Lines changed: 39 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -256,11 +256,48 @@ ecma_builtin_object_object_keys (ecma_value_t this_arg, /**< 'this' argument */
256256
* Returned value must be freed with ecma_free_completion_value.
257257
*/
258258
static ecma_completion_value_t
259-
ecma_builtin_object_object_get_own_property_descriptor (ecma_value_t this_arg, /**< 'this' argument */
259+
ecma_builtin_object_object_get_own_property_descriptor (ecma_value_t this_arg __attr_unused___, /**< 'this' argument */
260260
ecma_value_t arg1, /**< routine's first argument */
261261
ecma_value_t arg2) /**< routine's second argument */
262262
{
263-
ECMA_BUILTIN_CP_UNIMPLEMENTED (this_arg, arg1, arg2);
263+
ecma_completion_value_t ret_value = ecma_make_empty_completion_value ();
264+
265+
// 1.
266+
if (!ecma_is_value_object (arg1))
267+
{
268+
ret_value = ecma_make_throw_obj_completion_value (ecma_new_standard_error (ECMA_ERROR_TYPE));
269+
return ret_value;
270+
}
271+
272+
ecma_object_t *obj_p = ecma_get_object_from_value (arg1);
273+
274+
// 2.
275+
ECMA_TRY_CATCH (name_str_value,
276+
ecma_op_to_string (arg2),
277+
ret_value);
278+
279+
ecma_string_t *name_str_p = ecma_get_string_from_value (name_str_value);
280+
281+
// 3.
282+
ecma_property_t *prop_p = ecma_op_general_object_get_own_property (obj_p, name_str_p);
283+
284+
if (prop_p != NULL)
285+
{
286+
ecma_property_descriptor_t prop_desc = ecma_get_property_descriptor_from_property (prop_p);
287+
288+
// 4.
289+
ecma_object_t* desc_obj_p = ecma_op_from_property_descriptor (&prop_desc);
290+
291+
ret_value = ecma_make_normal_completion_value (ecma_make_object_value (desc_obj_p));
292+
}
293+
else
294+
{
295+
ret_value = ecma_make_simple_completion_value (ECMA_SIMPLE_VALUE_UNDEFINED);
296+
}
297+
298+
ECMA_FINALIZE (name_str_value);
299+
300+
return ret_value;
264301
} /* ecma_builtin_object_object_get_own_property_descriptor */
265302

266303
/**
Lines changed: 72 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,72 @@
1+
// Copyright 2015 Samsung Electronics Co., Ltd.
2+
// Copyright 2015 University of Szeged.
3+
//
4+
// Licensed under the Apache License, Version 2.0 (the "License");
5+
// you may not use this file except in compliance with the License.
6+
// You may obtain a copy of the License at
7+
//
8+
// http://www.apache.org/licenses/LICENSE-2.0
9+
//
10+
// Unless required by applicable law or agreed to in writing, software
11+
// distributed under the License is distributed on an "AS IS" BASIS
12+
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13+
// See the License for the specific language governing permissions and
14+
// limitations under the License.
15+
16+
var obj;
17+
var desc;
18+
19+
obj = { bar: 42 };
20+
desc = Object.getOwnPropertyDescriptor (obj, 'bar');
21+
// desc is { configurable: true, enumerable: true, value: 42, writable: true }
22+
assert (desc.value === 42);
23+
assert (desc.configurable);
24+
assert (desc.writable);
25+
assert (desc.enumerable);
26+
27+
obj = { "foo": "bar" };
28+
desc = Object.getOwnPropertyDescriptor (obj, 'foo');
29+
assert (desc.value === "bar");
30+
31+
obj = {};
32+
Object.defineProperty (obj, 'baz', { value: 8675309, writable: false, enumerable: false });
33+
desc = Object.getOwnPropertyDescriptor (obj, 'baz');
34+
// desc is { value: 8675309, writable: false, enumerable: false, configurable: false }
35+
assert (desc.value === 8675309);
36+
assert (!desc.configurable);
37+
assert (!desc.writable);
38+
assert (!desc.enumerable);
39+
40+
obj = { "0.00001": 1 };
41+
desc = Object.getOwnPropertyDescriptor (obj, 1e-5);
42+
assert ( desc.value === 1);
43+
44+
obj = { "123": 1 };
45+
desc = Object.getOwnPropertyDescriptor (obj, 123);
46+
assert (desc.value === 1);
47+
48+
obj = { "undefined": 1 };
49+
var desc1 = Object.getOwnPropertyDescriptor (obj, undefined);
50+
var desc2 = Object.getOwnPropertyDescriptor (obj, "undefined");
51+
assert (desc1.value === 1 && desc2.value === 1);
52+
53+
obj = { "0": 1 };
54+
desc = Object.getOwnPropertyDescriptor (obj, -0);
55+
assert (desc.value === 1);
56+
57+
obj = { "Hellobj": 42 };
58+
desc = Object.getOwnPropertyDescriptor (obj, new String ("Hellobj"));
59+
assert (desc.value === 42);
60+
61+
obj = {};
62+
desc = Object.getOwnPropertyDescriptor (obj, 'baz');
63+
assert (desc === undefined);
64+
65+
obj = { get foo() { return obj.bar; }, set foo(a) { obj.bar = a; }, bar: 0 };
66+
desc = Object.getOwnPropertyDescriptor(obj, 'foo');
67+
// d is { configurable: true, enumerable: true, get: /*the getter function*/, set: /*the setter function*/ }
68+
assert (typeof(desc.get) === 'function');
69+
assert (typeof(desc.set) === 'function');
70+
assert (desc.configurable);
71+
assert (desc.enumerable);
72+
assert (obj.foo === 0)

0 commit comments

Comments
 (0)