Unforked renderApplication()

Summary: This allows toggling fabric renderer via the same renderApplication()

Reviewed By: mdvacca

Differential Revision: D7682524

fbshipit-source-id: 59be1d2bea15f5e13e64e2d72304d79f9cb7d084

Libraries/ReactNative/AppContainer.js

@@ -27,7 +27,6 @@ type Props = {|
    * suppresses an error when upgrading Flow's support for React. To see the
    * error delete this comment and run Flow. */
   children?: React.Children,
-  fabric?: boolean,
   rootTag: number,
   WrapperComponent?: ?React.ComponentType<*>,
 |};
@@ -91,8 +90,7 @@ class AppContainer extends React.Component<Props, State> {
   render(): React.Node {
     let yellowBox = null;
     if (__DEV__) {
-      if (!global.__RCTProfileIsProfiling && !this.props.fabric) {
-        // TODO: Fabric doesn't support YellowBox.
+      if (!global.__RCTProfileIsProfiling) {
         const YellowBox = require('YellowBox');
         yellowBox = <YellowBox />;
       }

Libraries/ReactNative/AppRegistry.js

@@ -20,10 +20,6 @@ const infoLog = require('infoLog');
 const invariant = require('fbjs/lib/invariant');
 const renderApplication = require('renderApplication');
 
-// Renderer provider must be supplied by each app. If none, traditional
-// renderApplication() will be used.
-let fabricRendererProvider: ?() => typeof renderApplication = null;
-
 type Task = (taskData: any) => Promise<void>;
 type TaskProvider = () => Task;
 export type ComponentProvider = () => React$ComponentType<any>;
@@ -102,19 +98,12 @@ const AppRegistry = {
     runnables[appKey] = {
       componentProvider,
       run: appParameters => {
-        let renderFunc = renderApplication;
-        if (appParameters.fabric) {
-          invariant(
-            fabricRendererProvider != null,
-            'A Fabric renderer provider must be set to render Fabric components',
-          );
-          renderFunc = fabricRendererProvider();
-        }
-        renderFunc(
+        renderApplication(
           componentProviderInstrumentationHook(componentProvider),
           appParameters.initialProps,
           appParameters.rootTag,
           wrapperComponentProvider && wrapperComponentProvider(appParameters),
+          appParameters.fabric,
         );
       },
     };
@@ -249,10 +238,6 @@ const AppRegistry = {
         NativeModules.HeadlessJsTaskSupport.notifyTaskFinished(taskId);
       });
   },
-
-  setFabricRendererProvider(provider: () => typeof renderApplication): void {
-    fabricRendererProvider = provider;
-  },
 };
 
 BatchedBridge.registerCallableModule('AppRegistry', AppRegistry);

Libraries/ReactNative/renderApplication.js

@@ -13,6 +13,7 @@
 
 const AppContainer = require('AppContainer');
 const React = require('React');
+const ReactFabric = require('ReactFabric');
 const ReactNative = require('ReactNative');
 
 const invariant = require('fbjs/lib/invariant');
@@ -25,6 +26,7 @@ function renderApplication<Props: Object>(
   initialProps: Props,
   rootTag: any,
   WrapperComponent?: ?React.ComponentType<*>,
+  fabric?: boolean,
 ) {
   invariant(rootTag, 'Expect to have a valid rootTag, instead got ', rootTag);
 
@@ -49,7 +51,11 @@ function renderApplication<Props: Object>(
     renderable = <AsyncMode>{renderable}</AsyncMode>;
   }
 
-  ReactNative.render(renderable, rootTag);
+  if (fabric) {
+    ReactFabric.render(renderable, rootTag);
+  } else {
+    ReactNative.render(renderable, rootTag);
+  }
 }
 
 module.exports = renderApplication;