Add property key filters for built-in functions

jerry-core/api/jerry.c

@@ -4607,7 +4607,7 @@ jerry_object_get_property_names (const jerry_value_t obj_val, /**< object */
   while (true)
   {
     /* Step 1. Get Object.[[OwnKeys]] */
-    ecma_collection_t *prop_names_p = ecma_op_object_own_property_keys (obj_iter_p);
+    ecma_collection_t *prop_names_p = ecma_op_object_own_property_keys (obj_iter_p, filter);
 
 #if JERRY_BUILTIN_PROXY
     if (prop_names_p == NULL)

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

@@ -336,7 +336,7 @@ ecma_builtin_object_set_integrity_level (ecma_object_t *obj_p, /**< object */
   }
 
   /* 6. */
-  ecma_collection_t *props_p = ecma_op_object_own_property_keys (obj_p);
+  ecma_collection_t *props_p = ecma_op_object_own_property_keys (obj_p, JERRY_PROPERTY_FILTER_ALL);
 
 #if JERRY_BUILTIN_PROXY
   if (props_p == NULL)
@@ -596,7 +596,7 @@ ecma_builtin_object_test_integrity_level (ecma_object_t *obj_p, /**< routine's a
   ecma_value_t ret_value = ECMA_VALUE_TRUE;
 
   /* 2. */
-  ecma_collection_t *props_p = ecma_op_object_own_property_keys (obj_p);
+  ecma_collection_t *props_p = ecma_op_object_own_property_keys (obj_p, JERRY_PROPERTY_FILTER_ALL);
 
 #if JERRY_BUILTIN_PROXY
   if (props_p == NULL)
@@ -750,7 +750,7 @@ static ecma_value_t
 ecma_builtin_object_object_get_own_property_descriptors (ecma_object_t *obj_p) /**< routine's first argument */
 {
   /* 2 */
-  ecma_collection_t *prop_names_p = ecma_op_object_own_property_keys (obj_p);
+  ecma_collection_t *prop_names_p = ecma_op_object_own_property_keys (obj_p, JERRY_PROPERTY_FILTER_ALL);
 
 #if JERRY_BUILTIN_PROXY
   if (prop_names_p == NULL)
@@ -831,7 +831,7 @@ ecma_builtin_object_object_define_properties (ecma_object_t *obj_p, /**< routine
   ecma_object_t *props_p = ecma_get_object_from_value (props);
 
   /* 3. */
-  ecma_collection_t *prop_names_p = ecma_op_object_own_property_keys (props_p);
+  ecma_collection_t *prop_names_p = ecma_op_object_own_property_keys (props_p, JERRY_PROPERTY_FILTER_ALL);
   ecma_value_t ret_value = ECMA_VALUE_ERROR;
 
 #if JERRY_BUILTIN_PROXY
@@ -1073,7 +1073,7 @@ ecma_builtin_object_object_assign (ecma_object_t *target_p, /**< target object *
     ecma_object_t *from_obj_p = ecma_get_object_from_value (from_value);
 
     /* 5.b.iii */
-    ecma_collection_t *props_p = ecma_op_object_own_property_keys (from_obj_p);
+    ecma_collection_t *props_p = ecma_op_object_own_property_keys (from_obj_p, JERRY_PROPERTY_FILTER_ALL);
 
 #if JERRY_BUILTIN_PROXY
     if (props_p == NULL)
@@ -1327,7 +1327,15 @@ ecma_op_object_get_own_property_keys (ecma_value_t this_arg, /**< this argument
   ecma_object_t *obj_p = ecma_get_object_from_value (object);
 
   /* 2. */
-  ecma_collection_t *props_p = ecma_op_object_own_property_keys (obj_p);
+  jerry_property_filter_t filter = JERRY_PROPERTY_FILTER_EXLCUDE_SYMBOLS;
+
+  if (type == ECMA_OBJECT_ROUTINE_GET_OWN_PROPERTY_SYMBOLS)
+  {
+    filter = (JERRY_PROPERTY_FILTER_EXLCUDE_STRINGS
+              | JERRY_PROPERTY_FILTER_EXLCUDE_INTEGER_INDICES);
+  }
+
+  ecma_collection_t *props_p = ecma_op_object_own_property_keys (obj_p, filter);
 
   if (props_p == NULL)
   {
@@ -1361,7 +1369,7 @@ ecma_op_object_get_own_property_keys (ecma_value_t this_arg, /**< this argument
 #else /* !JERRY_ESNEXT */
   JERRY_UNUSED (type);
   ecma_object_t *obj_p = ecma_get_object_from_value (this_arg);
-  ecma_collection_t *props_p = ecma_op_object_own_property_keys (obj_p);
+  ecma_collection_t *props_p = ecma_op_object_own_property_keys (obj_p, JERRY_PROPERTY_FILTER_ALL);
   return ecma_op_new_array_object_from_collection (props_p, false);
 #endif /* JERRY_ESNEXT */
 } /* ecma_op_object_get_own_property_keys */

jerry-core/ecma/builtin-objects/ecma-builtin-reflect.c

@@ -163,7 +163,7 @@ ecma_builtin_reflect_dispatch_routine (uint8_t builtin_routine_id, /**< built-in
     ecma_object_t *target_p = ecma_get_object_from_value (arguments_list[0]);
 
     /* 2. */
-    ecma_collection_t *prop_names = ecma_op_object_own_property_keys (target_p);
+    ecma_collection_t *prop_names = ecma_op_object_own_property_keys (target_p, JERRY_PROPERTY_FILTER_ALL);
 
 #if JERRY_BUILTIN_PROXY
     if (prop_names == NULL)

jerry-core/ecma/builtin-objects/ecma-builtins.c

@@ -1414,11 +1414,17 @@ ecma_builtin_native_handler_list_lazy_property_names (ecma_object_t *object_p, /
 void
 ecma_builtin_routine_list_lazy_property_names (ecma_object_t *object_p, /**< a built-in object */
                                                ecma_collection_t *prop_names_p,  /**< prop name collection */
-                                               ecma_property_counter_t *prop_counter_p)  /**< prop counter */
+                                               ecma_property_counter_t *prop_counter_p, /**< property counters */
+                                               jerry_property_filter_t filter) /**< name filters */
 {
   JERRY_ASSERT (ecma_get_object_type (object_p) == ECMA_OBJECT_TYPE_BUILT_IN_FUNCTION);
   JERRY_ASSERT (ecma_builtin_function_is_routine (object_p));
 
+  if (filter & JERRY_PROPERTY_FILTER_EXLCUDE_STRINGS)
+  {
+    return;
+  }
+
 #if JERRY_ESNEXT
   ecma_extended_object_t *ext_func_p = (ecma_extended_object_t *) object_p;
 
@@ -1456,7 +1462,8 @@ ecma_builtin_routine_list_lazy_property_names (ecma_object_t *object_p, /**< a b
 void
 ecma_builtin_list_lazy_property_names (ecma_object_t *object_p, /**< a built-in object */
                                        ecma_collection_t *prop_names_p, /**< prop name collection */
-                                       ecma_property_counter_t *prop_counter_p)  /**< prop counter */
+                                       ecma_property_counter_t *prop_counter_p, /**< property counters */
+                                       jerry_property_filter_t filter) /**< name filters */
 {
   JERRY_ASSERT (ecma_get_object_type (object_p) != ECMA_OBJECT_TYPE_BUILT_IN_FUNCTION
                 || !ecma_builtin_function_is_routine (object_p));
@@ -1477,57 +1484,91 @@ ecma_builtin_list_lazy_property_names (ecma_object_t *object_p, /**< a built-in
 
   JERRY_ASSERT (builtin_id < ECMA_BUILTIN_ID__COUNT);
 
-  const ecma_builtin_property_descriptor_t *curr_property_p = ecma_builtin_property_list_references[builtin_id];
-
-  uint32_t index = 0;
-  uint8_t bitset = built_in_props_p->u2.instantiated_bitset[0];
-
 #if JERRY_BUILTIN_REALMS
   uint8_t *bitset_p = built_in_props_p->u2.instantiated_bitset + 1 + sizeof (ecma_value_t);
 #else /* !JERRY_BUILTIN_REALMS */
   uint8_t *bitset_p = built_in_props_p->u2.instantiated_bitset + 1;
 #endif /* JERRY_BUILTIN_REALMS */
 
-  while (curr_property_p->magic_string_id != LIT_MAGIC_STRING__COUNT)
+#if JERRY_ESNEXT
+  uint8_t *symbol_bitset_p = bitset_p;
+  bool has_symbol = true;
+#endif /* JERRY_BUILTIN_REALMS */
+
+  if (!(filter & JERRY_PROPERTY_FILTER_EXLCUDE_STRINGS))
   {
-    if (index == 8)
-    {
-      bitset = *bitset_p++;
-      index = 0;
-    }
+    const ecma_builtin_property_descriptor_t *curr_property_p = ecma_builtin_property_list_references[builtin_id];
+    uint8_t bitset = built_in_props_p->u2.instantiated_bitset[0];
+    uint32_t index = 0;
 
-    uint32_t bit_for_index = (uint32_t) 1u << index;
+#if JERRY_ESNEXT
+    has_symbol = false;
+#endif /* JERRY_BUILTIN_REALMS */
 
-    if (curr_property_p->magic_string_id > LIT_NON_INTERNAL_MAGIC_STRING__COUNT)
+    while (curr_property_p->magic_string_id != LIT_MAGIC_STRING__COUNT)
     {
-#if JERRY_ESNEXT
-      if (LIT_IS_GLOBAL_SYMBOL (curr_property_p->magic_string_id))
+      if (index == 8)
       {
-        ecma_string_t *name_p = ecma_op_get_global_symbol (curr_property_p->magic_string_id);
+        bitset = *bitset_p++;
+        index = 0;
+      }
+
+      uint32_t bit_for_index = (uint32_t) 1u << index;
 
-        if (!(bitset & bit_for_index))
+      if (!(bitset & bit_for_index))
+      {
+#if JERRY_ESNEXT
+        if (JERRY_LIKELY (curr_property_p->magic_string_id < LIT_NON_INTERNAL_MAGIC_STRING__COUNT))
         {
-          ecma_value_t name = ecma_make_symbol_value (name_p);
+#endif /* JERRY_ESNEXT */
+          ecma_value_t name = ecma_make_magic_string_value ((lit_magic_string_id_t) curr_property_p->magic_string_id);
           ecma_collection_push_back (prop_names_p, name);
-          prop_counter_p->symbol_named_props++;
+          prop_counter_p->string_named_props++;
+#if JERRY_ESNEXT
         }
         else
         {
-          ecma_deref_ecma_string (name_p);
+          JERRY_ASSERT (LIT_IS_GLOBAL_SYMBOL (curr_property_p->magic_string_id));
+          has_symbol = true;
         }
-      }
 #endif /* JERRY_ESNEXT */
+      }
+
+      curr_property_p++;
+      index++;
     }
-    else if (!(bitset & bit_for_index))
+  }
+
+#if JERRY_ESNEXT
+  if (has_symbol && !(filter & JERRY_PROPERTY_FILTER_EXLCUDE_SYMBOLS))
+  {
+    const ecma_builtin_property_descriptor_t *curr_property_p = ecma_builtin_property_list_references[builtin_id];
+    uint8_t bitset = built_in_props_p->u2.instantiated_bitset[0];
+    uint32_t index = 0;
+
+    while (curr_property_p->magic_string_id != LIT_MAGIC_STRING__COUNT)
     {
-      ecma_value_t name = ecma_make_magic_string_value ((lit_magic_string_id_t) curr_property_p->magic_string_id);
-      ecma_collection_push_back (prop_names_p, name);
-      prop_counter_p->string_named_props++;
-    }
+      if (index == 8)
+      {
+        bitset = *symbol_bitset_p++;
+        index = 0;
+      }
 
-    curr_property_p++;
-    index++;
+      uint32_t bit_for_index = (uint32_t) 1u << index;
+
+      if (curr_property_p->magic_string_id > LIT_NON_INTERNAL_MAGIC_STRING__COUNT
+          && !(bitset & bit_for_index))
+      {
+        ecma_string_t *name_p = ecma_op_get_global_symbol (curr_property_p->magic_string_id);
+        ecma_collection_push_back (prop_names_p, ecma_make_symbol_value (name_p));
+        prop_counter_p->symbol_named_props++;
+      }
+
+      curr_property_p++;
+      index++;
+    }
   }
+#endif /* JERRY_ESNEXT */
 } /* ecma_builtin_list_lazy_property_names */
 
 /**

jerry-core/ecma/builtin-objects/ecma-builtins.h

@@ -138,13 +138,13 @@ ecma_builtin_routine_delete_built_in_property (ecma_object_t *object_p, ecma_str
 void
 ecma_builtin_delete_built_in_property (ecma_object_t *object_p, ecma_string_t *property_name_p);
 void
-ecma_builtin_routine_list_lazy_property_names (ecma_object_t *object_p,
-                                               ecma_collection_t *prop_names_p,
-                                               ecma_property_counter_t *prop_counter_p);
+ecma_builtin_routine_list_lazy_property_names (ecma_object_t *object_p, ecma_collection_t *prop_names_p,
+                                               ecma_property_counter_t *prop_counter_p,
+                                               jerry_property_filter_t filter);
 void
-ecma_builtin_list_lazy_property_names (ecma_object_t *object_p,
-                                       ecma_collection_t *prop_names_p,
-                                       ecma_property_counter_t *prop_counter_p);
+ecma_builtin_list_lazy_property_names (ecma_object_t *object_p, ecma_collection_t *prop_names_p,
+                                       ecma_property_counter_t *prop_counter_p,
+                                       jerry_property_filter_t filter);
 bool
 ecma_builtin_is_global (ecma_object_t *object_p);
 ecma_object_t *

jerry-core/ecma/operations/ecma-arguments-object.c

@@ -401,7 +401,8 @@ ecma_op_arguments_delete_built_in_property (ecma_object_t *object_p, /**< the ob
 void
 ecma_op_arguments_object_list_lazy_property_names (ecma_object_t *obj_p, /**< arguments object */
                                                    ecma_collection_t *prop_names_p, /**< prop name collection */
-                                                   ecma_property_counter_t *prop_counter_p) /**< property counters */
+                                                   ecma_property_counter_t *prop_counter_p, /**< property counters */
+                                                   jerry_property_filter_t filter) /**< property name filter options */
 {
   JERRY_ASSERT (ecma_object_class_is (obj_p, ECMA_OBJECT_CLASS_ARGUMENTS));
 
@@ -410,45 +411,52 @@ ecma_op_arguments_object_list_lazy_property_names (ecma_object_t *obj_p, /**< ar
   uint32_t arguments_number = arguments_p->header.u.cls.u3.arguments_number;
   uint8_t flags = arguments_p->header.u.cls.u1.arguments_flags;
 
-  ecma_value_t *argv_p = (ecma_value_t *) (arguments_p + 1);
-
-  if (flags & ECMA_ARGUMENTS_OBJECT_MAPPED)
+  if (!(filter & JERRY_PROPERTY_FILTER_EXLCUDE_INTEGER_INDICES))
   {
-    argv_p = (ecma_value_t *) (((ecma_mapped_arguments_t *) obj_p) + 1);
-  }
+    ecma_value_t *argv_p = (ecma_value_t *) (arguments_p + 1);
 
-  for (uint32_t index = 0; index < arguments_number; index++)
-  {
-    if (!ecma_is_value_empty (argv_p[index]))
+    if (flags & ECMA_ARGUMENTS_OBJECT_MAPPED)
     {
-      ecma_string_t *index_string_p = ecma_new_ecma_string_from_uint32 (index);
-      ecma_collection_push_back (prop_names_p, ecma_make_string_value (index_string_p));
-      prop_counter_p->array_index_named_props++;
+      argv_p = (ecma_value_t *) (((ecma_mapped_arguments_t *) obj_p) + 1);
     }
-  }
 
-  if (!(flags & ECMA_ARGUMENTS_OBJECT_LENGTH_INITIALIZED))
-  {
-    ecma_collection_push_back (prop_names_p, ecma_make_magic_string_value (LIT_MAGIC_STRING_LENGTH));
-    prop_counter_p->string_named_props++;
+    for (uint32_t index = 0; index < arguments_number; index++)
+    {
+      if (!ecma_is_value_empty (argv_p[index]))
+      {
+        ecma_string_t *index_string_p = ecma_new_ecma_string_from_uint32 (index);
+        ecma_collection_push_back (prop_names_p, ecma_make_string_value (index_string_p));
+        prop_counter_p->array_index_named_props++;
+      }
+    }
   }
 
-  if (!(flags & ECMA_ARGUMENTS_OBJECT_CALLEE_INITIALIZED))
+  if (!(filter & JERRY_PROPERTY_FILTER_EXLCUDE_STRINGS))
   {
-    ecma_collection_push_back (prop_names_p, ecma_make_magic_string_value (LIT_MAGIC_STRING_CALLEE));
-    prop_counter_p->string_named_props++;
-  }
+    if (!(flags & ECMA_ARGUMENTS_OBJECT_LENGTH_INITIALIZED))
+    {
+      ecma_collection_push_back (prop_names_p, ecma_make_magic_string_value (LIT_MAGIC_STRING_LENGTH));
+      prop_counter_p->string_named_props++;
+    }
+
+    if (!(flags & ECMA_ARGUMENTS_OBJECT_CALLEE_INITIALIZED))
+    {
+      ecma_collection_push_back (prop_names_p, ecma_make_magic_string_value (LIT_MAGIC_STRING_CALLEE));
+      prop_counter_p->string_named_props++;
+    }
 
 #if !JERRY_ESNEXT
-  if (!(flags & ECMA_ARGUMENTS_OBJECT_MAPPED))
-  {
-    ecma_collection_push_back (prop_names_p, ecma_make_magic_string_value (LIT_MAGIC_STRING_CALLER));
-    prop_counter_p->string_named_props++;
-  }
+    if (!(flags & ECMA_ARGUMENTS_OBJECT_MAPPED))
+    {
+      ecma_collection_push_back (prop_names_p, ecma_make_magic_string_value (LIT_MAGIC_STRING_CALLER));
+      prop_counter_p->string_named_props++;
+    }
 #endif /* !JERRY_ESNEXT */
+  }
 
 #if JERRY_ESNEXT
-  if (!(flags & ECMA_ARGUMENTS_OBJECT_ITERATOR_INITIALIZED))
+  if (!(filter & JERRY_PROPERTY_FILTER_EXLCUDE_SYMBOLS)
+      && !(flags & ECMA_ARGUMENTS_OBJECT_ITERATOR_INITIALIZED))
   {
     ecma_string_t *symbol_p = ecma_op_get_global_symbol (LIT_GLOBAL_SYMBOL_ITERATOR);
     ecma_collection_push_back (prop_names_p, ecma_make_symbol_value (symbol_p));

jerry-core/ecma/operations/ecma-arguments-object.h

@@ -36,7 +36,8 @@ ecma_op_arguments_delete_built_in_property (ecma_object_t *object_p, ecma_string
 
 void
 ecma_op_arguments_object_list_lazy_property_names (ecma_object_t *obj_p, ecma_collection_t *prop_names_p,
-                                                   ecma_property_counter_t *prop_counter_p);
+                                                   ecma_property_counter_t *prop_counter_p,
+                                                   jerry_property_filter_t filter);
 
 ecma_string_t *
 ecma_op_arguments_object_get_formal_parameter (ecma_mapped_arguments_t *mapped_arguments_p,