Do not initialize lazy data properties in v8::Object::GetPropertyNames().

Author: iamstolis

graal-nodejs/mx.graal-nodejs/com.oracle.truffle.trufflenode/src/com/oracle/truffle/trufflenode/GraalJSAccess.java

@@ -159,6 +159,7 @@
 import com.oracle.truffle.api.nodes.UnexpectedResultException;
 import com.oracle.truffle.api.object.DynamicObjectLibrary;
 import com.oracle.truffle.api.object.HiddenKey;
+import com.oracle.truffle.api.object.Property;
 import com.oracle.truffle.api.source.Source;
 import com.oracle.truffle.api.source.SourceSection;
 import com.oracle.truffle.api.strings.TruffleString;
@@ -255,6 +256,7 @@
 import com.oracle.truffle.js.runtime.objects.JSObject;
 import com.oracle.truffle.js.runtime.objects.JSObjectUtil;
 import com.oracle.truffle.js.runtime.objects.JSOrdinaryObject;
+import com.oracle.truffle.js.runtime.objects.JSProperty;
 import com.oracle.truffle.js.runtime.objects.Null;
 import com.oracle.truffle.js.runtime.objects.PromiseCapabilityRecord;
 import com.oracle.truffle.js.runtime.objects.PropertyDescriptor;
@@ -1157,6 +1159,7 @@ public Object objectGetPropertyNames(Object object, boolean ownOnly,
             JSDynamicObject dynamicObject = (JSDynamicObject) object;
             do {
                 JSClass jsclass = JSObject.getJSClass(dynamicObject);
+                boolean ordinaryGetOwnProperty = jsclass.usesOrdinaryGetOwnProperty();
                 Iterable<Object> ownKeys = jsclass.ownPropertyKeys(dynamicObject);
                 for (Object key : ownKeys) {
                     Object keyToStore = key;
@@ -1179,10 +1182,18 @@ public Object objectGetPropertyNames(Object object, boolean ownOnly,
                             continue;
                         }
                     }
-                    PropertyDescriptor desc = jsclass.getOwnProperty(dynamicObject, key);
-                    if ((enumerableOnly && (desc == null || !desc.getEnumerable())) || (configurableOnly && (desc == null || !desc.getConfigurable())) ||
-                                    (writableOnly && (desc == null || !desc.getWritable()))) {
-                        continue;
+                    if (ordinaryGetOwnProperty) {
+                        Property prop = dynamicObject.getShape().getProperty(key);
+                        if ((enumerableOnly && (prop == null || !JSProperty.isEnumerable(prop))) || (configurableOnly && (prop == null || !JSProperty.isConfigurable(prop))) ||
+                                        (writableOnly && (prop == null || !JSProperty.isWritable(prop)))) {
+                            continue;
+                        }
+                    } else {
+                        PropertyDescriptor desc = jsclass.getOwnProperty(dynamicObject, key);
+                        if ((enumerableOnly && (desc == null || !desc.getEnumerable())) || (configurableOnly && (desc == null || !desc.getConfigurable())) ||
+                                        (writableOnly && (desc == null || !desc.getWritable()))) {
+                            continue;
+                        }
                     }
                     keys.add(keyToStore);
                 }