Remove Reflect.enumerate

src/bootstrapper.cc

Index: src/bootstrapper.cc
diff --git a/src/bootstrapper.cc b/src/bootstrapper.cc
index 3d30cced7ac0135095aa160a67df3bae61b26e31..59af448c2d18f75a56df988e8c4c28ee424c1bfd 100644
--- a/src/bootstrapper.cc
+++ b/src/bootstrapper.cc
@@ -2420,6 +2420,10 @@ void Genesis::InitializeGlobal_harmony_reflect() {
   InstallFunction(reflect, define_property, factory->defineProperty_string());
   InstallFunction(reflect, delete_property, factory->deleteProperty_string());
 
+  SimpleInstallFunction(reflect, factory->apply_string(),

[adamk, 2016/02/20 00:06:14]

Hmm, seems like this is doing the same thing as the bits about reflect_apply and
reflect_construct over in ExportFromRuntime. Can we just do the work once?

[Dan Ehrenberg, 2016/02/20 00:12:28]

The parser has desugarings into calls to reflect_apply and reflect_construct, so
I don't think I can just remove them. However, adding these two lines means that
I can remove the harmony-reflect.js file.

[adamk, 2016/02/20 00:23:55]

Take a look at how object_is_extensible is handled, I think you ought to be able
to do that for these ones. In short, you capture the return value of this
SimpleInstallFunction, then set it in the native_context slot, and then change
the JS callers to use a '%' prefix when calling it and stop importing.

[Dan Ehrenberg, 2016/02/20 00:58:55]

Thanks for the reference; adopted that pattern. PTAL.

+                        Builtins::kReflectApply, 3, false);
+  SimpleInstallFunction(reflect, factory->construct_string(),
+                        Builtins::kReflectConstruct, 2, false);
   SimpleInstallFunction(reflect, factory->get_string(),
                         Builtins::kReflectGet, 2, false);
   SimpleInstallFunction(reflect, factory->getOwnPropertyDescriptor_string(),