[proxies] Use internalized strings for trap names.

src/bootstrapper.cc

 // Copyright 2014 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.
 
 #include "src/bootstrapper.h"
 
 #include "src/accessors.h"
 #include "src/api-natives.h"
 #include "src/code-stubs.h"
 #include "src/extensions/externalize-string-extension.h"
@@ skipping 2047 matching lines @@
 EMPTY_INITIALIZE_GLOBAL_FOR_FEATURE(harmony_destructuring_bind)
 EMPTY_INITIALIZE_GLOBAL_FOR_FEATURE(harmony_object_observe)
 EMPTY_INITIALIZE_GLOBAL_FOR_FEATURE(harmony_regexps)
 EMPTY_INITIALIZE_GLOBAL_FOR_FEATURE(harmony_unicode_regexps)
 EMPTY_INITIALIZE_GLOBAL_FOR_FEATURE(harmony_completion)
 EMPTY_INITIALIZE_GLOBAL_FOR_FEATURE(harmony_tolength)
 EMPTY_INITIALIZE_GLOBAL_FOR_FEATURE(harmony_do_expressions)
 EMPTY_INITIALIZE_GLOBAL_FOR_FEATURE(harmony_regexp_lookbehind)
 
 
-static void SimpleInstallFunction(Handle<JSObject>& base, const char* name,
+static void SimpleInstallFunction(Handle<JSObject> base, Handle<Name> name,
                                   Builtins::Name call, int len, bool adapt) {
   Handle<JSFunction> fun =
       InstallFunction(base, name, JS_OBJECT_TYPE, JSObject::kHeaderSize,
-                      MaybeHandle<JSObject>(), call);
+                      MaybeHandle<JSObject>(), call, DONT_ENUM);
   if (adapt) {
     fun->shared()->set_internal_formal_parameter_count(len);
   } else {
     fun->shared()->DontAdaptArguments();
   }
   fun->shared()->set_length(len);
 }
 
 
 void InstallPublicSymbol(Factory* factory, Handle<Context> native_context,
@@ skipping 33 matching lines @@
   InstallPublicSymbol(factory(), native_context(), "search",
                       factory()->search_symbol());
   InstallPublicSymbol(factory(), native_context(), "split",
                       factory()->split_symbol());
 }
 
 
 void Genesis::InitializeGlobal_harmony_reflect() {
   if (!FLAG_harmony_reflect) return;
 
+  Factory* factory = isolate()->factory();
   Handle<JSGlobalObject> global(JSGlobalObject::cast(
       native_context()->global_object()));
-  Handle<String> reflect_string =
-      factory()->NewStringFromStaticChars("Reflect");
+  Handle<String> reflect_string = factory->NewStringFromStaticChars("Reflect");
   Handle<JSObject> reflect =
-      factory()->NewJSObject(isolate()->object_function(), TENURED);
+      factory->NewJSObject(isolate()->object_function(), TENURED);
   JSObject::AddProperty(global, reflect_string, reflect, DONT_ENUM);
 
-  SimpleInstallFunction(reflect, "defineProperty",
+  SimpleInstallFunction(reflect, factory->defineProperty_string(),
                         Builtins::kReflectDefineProperty, 3, true);
-  SimpleInstallFunction(reflect, "deleteProperty",
+  SimpleInstallFunction(reflect, factory->deleteProperty_string(),
                         Builtins::kReflectDeleteProperty, 2, true);
-  SimpleInstallFunction(reflect, "get",
+  SimpleInstallFunction(reflect, factory->get_string(),
                         Builtins::kReflectGet, 3, false);
-  SimpleInstallFunction(reflect, "getOwnPropertyDescriptor",
+  SimpleInstallFunction(reflect, factory->getOwnPropertyDescriptor_string(),
                         Builtins::kReflectGetOwnPropertyDescriptor, 2, true);
-  SimpleInstallFunction(reflect, "getPrototypeOf",
+  SimpleInstallFunction(reflect, factory->getPrototypeOf_string(),
                         Builtins::kReflectGetPrototypeOf, 1, true);
-  SimpleInstallFunction(reflect, "has",
+  SimpleInstallFunction(reflect, factory->has_string(),
                         Builtins::kReflectHas, 2, true);
-  SimpleInstallFunction(reflect, "isExtensible",
+  SimpleInstallFunction(reflect, factory->isExtensible_string(),
                         Builtins::kReflectIsExtensible, 1, true);
-  SimpleInstallFunction(reflect, "ownKeys",
+  SimpleInstallFunction(reflect, factory->ownKeys_string(),
                         Builtins::kReflectOwnKeys, 1, true);
-  SimpleInstallFunction(reflect, "preventExtensions",
+  SimpleInstallFunction(reflect, factory->preventExtensions_string(),
                         Builtins::kReflectPreventExtensions, 1, true);
-  SimpleInstallFunction(reflect, "set",
+  SimpleInstallFunction(reflect, factory->set_string(),
                         Builtins::kReflectSet, 3, false);
-  SimpleInstallFunction(reflect, "setPrototypeOf",
+  SimpleInstallFunction(reflect, factory->setPrototypeOf_string(),
                         Builtins::kReflectSetPrototypeOf, 2, true);
 }
 
 
 void Genesis::InitializeGlobal_harmony_sharedarraybuffer() {
   if (!FLAG_harmony_sharedarraybuffer) return;
 
   Handle<JSGlobalObject> global(
       JSGlobalObject::cast(native_context()->global_object()));
 
@@ skipping 1112 matching lines @@
 }
 
 
 // Called when the top-level V8 mutex is destroyed.
 void Bootstrapper::FreeThreadResources() {
   DCHECK(!IsActive());
 }
 
 }  // namespace internal
 }  // namespace v8