[proxies] Use JSReceiver::GetKeys() for more purposes

src/js/symbol.js

 // Copyright 2013 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 (function(global, utils) {
 
 "use strict";
 
 %CheckIsBootstrapping();
 
 // -------------------------------------------------------------------
 // Imports
 
 var GlobalObject = global.Object;
 var GlobalSymbol = global.Symbol;
 var hasInstanceSymbol = utils.ImportNow("has_instance_symbol");
 var isConcatSpreadableSymbol =
     utils.ImportNow("is_concat_spreadable_symbol");
 var iteratorSymbol = utils.ImportNow("iterator_symbol");
 var MakeTypeError;
-var ObjectGetOwnPropertyKeys;
 var toPrimitiveSymbol = utils.ImportNow("to_primitive_symbol");
 var toStringTagSymbol = utils.ImportNow("to_string_tag_symbol");
 var unscopablesSymbol = utils.ImportNow("unscopables_symbol");
 
 utils.Import(function(from) {
   MakeTypeError = from.MakeTypeError;
-  ObjectGetOwnPropertyKeys = from.ObjectGetOwnPropertyKeys;
 });
 
 // -------------------------------------------------------------------
 
 // 19.4.3.4 Symbol.prototype [ @@toPrimitive ] ( hint )
 function SymbolToPrimitive(hint) {
   if (!(IS_SYMBOL(this) || IS_SYMBOL_WRAPPER(this))) {
     throw MakeTypeError(kIncompatibleMethodReceiver,
                         "Symbol.prototype [ @@toPrimitive ]", this);
   }
@@ skipping 34 matching lines @@
 function SymbolKeyFor(symbol) {
   if (!IS_SYMBOL(symbol)) throw MakeTypeError(kSymbolKeyFor, symbol);
   return %SymbolRegistry().keyFor[symbol];
 }
 
 
 // ES6 19.1.2.8
 function ObjectGetOwnPropertySymbols(obj) {
   obj = TO_OBJECT(obj);
 
-  // TODO(arv): Proxies use a shared trap for String and Symbol keys.
-
-  return ObjectGetOwnPropertyKeys(obj, PROPERTY_ATTRIBUTES_STRING);
+  return %GetOwnPropertyKeys(obj, PROPERTY_FILTER_SKIP_STRINGS);
 }
 
 // -------------------------------------------------------------------
 
 %FunctionSetPrototype(GlobalSymbol, new GlobalObject());
 
 utils.InstallConstants(GlobalSymbol, [
   // TODO(rossberg): expose when implemented.
   // "hasInstance", hasInstanceSymbol,
   // "isConcatSpreadable", isConcatSpreadableSymbol,
@@ skipping 34 matching lines @@
 ]);
 
 // -------------------------------------------------------------------
 // Exports
 
 utils.Export(function(to) {
   to.SymbolToString = SymbolToString;
 })
 
 })