Don't bundle ReactComponentTreeHook in production (#7653)

Fixes #7492. This was a build size regression introduced in #7164.
facebook · Sep 3, 2016 · a09d158 · a09d158
1 parent 9d9af63
commit a09d158
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 2 deletions.
diff --git a/src/umd/ReactUMDEntry.js b/src/umd/ReactUMDEntry.js
@@ -17,8 +17,17 @@ var React = require('React');
 var ReactUMDEntry = Object.assign({
   __SECRET_INTERNALS_DO_NOT_USE_OR_YOU_WILL_BE_FIRED: {
     ReactCurrentOwner: require('ReactCurrentOwner'),
-    ReactComponentTreeHook: require('ReactComponentTreeHook'),
   },
 }, React);
 
+if (__DEV__) {
+  Object.assign(
+    ReactUMDEntry.__SECRET_INTERNALS_DO_NOT_USE_OR_YOU_WILL_BE_FIRED,
+    {
+      // ReactComponentTreeHook should not be included in production.
+      ReactComponentTreeHook: require('ReactComponentTreeHook'),
+    }
+  );
+}
+
 module.exports = ReactUMDEntry;
diff --git a/src/umd/ReactWithAddonsUMDEntry.js b/src/umd/ReactWithAddonsUMDEntry.js
@@ -17,8 +17,17 @@ var ReactWithAddons = require('ReactWithAddons');
 var ReactWithAddonsUMDEntry = Object.assign({
   __SECRET_INTERNALS_DO_NOT_USE_OR_YOU_WILL_BE_FIRED: {
     ReactCurrentOwner: require('ReactCurrentOwner'),
-    ReactComponentTreeHook: require('ReactComponentTreeHook'),
   },
 }, ReactWithAddons);
 
+if (__DEV__) {
+  Object.assign(
+    ReactWithAddonsUMDEntry.__SECRET_INTERNALS_DO_NOT_USE_OR_YOU_WILL_BE_FIRED,
+    {
+      // ReactComponentTreeHook should not be included in production.
+      ReactComponentTreeHook: require('ReactComponentTreeHook'),
+    }
+  );
+}
+
 module.exports = ReactWithAddonsUMDEntry;