Include symbol properties in Object.{create,defineProperties}

src/v8natives.js

Index: src/v8natives.js
diff --git a/src/v8natives.js b/src/v8natives.js
index 5ffff2ecacd9f789d475595aa506c3093d20bfe4..14c6c1b4eff2eea2e4e3687fc910716eb9567551 100644
--- a/src/v8natives.js
+++ b/src/v8natives.js
@@ -1171,13 +1171,24 @@ function ObjectDefineProperty(obj, p, attributes) {
 }
 
 
-function GetOwnEnumerablePropertyNames(properties) {
+function GetOwnEnumerablePropertyNames(object) {
   var names = new InternalArray();
-  for (var key in properties) {
-    if (%HasOwnProperty(properties, key)) {
+  for (var key in object) {
+    if (%HasOwnProperty(object, key)) {
       names.push(key);
     }
   }
+  // FLAG_harmony_symbols may be on, but symbols aren't included by for-in.
+  var symbols = ObjectGetOwnPropertyKeys(object, true);
+  for (var i in symbols) {
+    var symbol = symbols[i];
+    %DebugPrint(symbol);

[Michael Starzinger, 2014/07/14 10:56:01]

nit: This looks like a leftover.

+    if (ObjectGetOwnPropertyDescriptor(object, symbol).enumerable) {
+      %DebugPrint('yes');

[Michael Starzinger, 2014/07/14 10:56:01]

nit: This looks like a leftover.

+      names.push(symbol);
+    }
+    else %DebugPrint('no');

[Michael Starzinger, 2014/07/14 10:56:01]

nit: This looks like a leftover.

+  }
   return names;
 }