Use NoAutomaticPrototype everywhere

Reviewed By: michalgr

Differential Revision: D4462794

fbshipit-source-id: 53585741c1d35ca31e3429f2de56455ea5dea902

Author: javache

ReactCommon/cxxreact/JSCExecutor.cpp

@@ -240,7 +240,9 @@ void JSCExecutor::initOnJSVMThread() throw(JSException) {
   JSClassRef globalClass = nullptr;
   {
     SystraceSection s("JSClassCreate");
-    globalClass = JSC_JSClassCreate(useCustomJSC, &kJSClassDefinitionEmpty);
+    JSClassDefinition definition = kJSClassDefinitionEmpty;
+    definition.attributes |= kJSClassAttributeNoAutomaticPrototype;
+    globalClass = JSC_JSClassCreate(useCustomJSC, &definition);
   }
   {
     SystraceSection s("JSGlobalContextCreateInGroup");

ReactCommon/jschelpers/JSCHelpers.cpp

@@ -30,7 +30,8 @@ JSValueRef functionCaller(
 }
 
 JSClassRef createFuncClass(JSContextRef ctx) {
-  auto definition = kJSClassDefinitionEmpty;
+  JSClassDefinition definition = kJSClassDefinitionEmpty;
+  definition.attributes |= kJSClassAttributeNoAutomaticPrototype;
   // Need to duplicate the two different finalizer blocks, since there's no way
   // for it to capture this static information.
   if (isCustomJSCPtr(ctx)) {
@@ -107,7 +108,7 @@ void installGlobalProxy(
     const char* name,
     JSObjectGetPropertyCallback callback) {
   JSClassDefinition proxyClassDefintion = kJSClassDefinitionEmpty;
-  proxyClassDefintion.className = "_FBProxyClass";
+  proxyClassDefintion.attributes |= kJSClassAttributeNoAutomaticPrototype;
   proxyClassDefintion.getProperty = callback;
 
   const bool isCustomJSC = isCustomJSCPtr(ctx);