Teach JSReceiver::GetKeys() how to include symbols

src/objects.h

Index: src/objects.h
diff --git a/src/objects.h b/src/objects.h
index 225a7db42e0cfab92c786cea07095731304ffa22..6a88b0e3dc6ebb5f0a281c75ff9fa95ed37064c2 100644
--- a/src/objects.h
+++ b/src/objects.h
@@ -1850,12 +1850,13 @@ class JSReceiver: public HeapObject {
       Handle<JSReceiver> object);
 
   enum KeyCollectionType { OWN_ONLY, INCLUDE_PROTOS };
+  enum IncludeSymbolNamedProperties { SKIP_SYMBOLS, INCLUDE_SYMBOLS };

[Camillo Bruni, 2015/09/29 16:27:20]

Maybe use FixedArray::NON_SYMBOL_KEYS instead (or move the enum from FixedArray
to the global space or JSReceiver);

[Jakob Kummerow, 2015/09/30 13:59:30]

Done.

 
   // Computes the enumerable keys for a JSObject. Used for implementing
   // "for (n in object) { }".
   MUST_USE_RESULT static MaybeHandle<FixedArray> GetKeys(
-      Handle<JSReceiver> object,
-      KeyCollectionType type);
+      Handle<JSReceiver> object, KeyCollectionType type,
+      IncludeSymbolNamedProperties include_symbols = SKIP_SYMBOLS);
 
  private:
   DISALLOW_IMPLICIT_CONSTRUCTORS(JSReceiver);