revert "Disable DebugTools in production (facebook#7189)"

keyz · Jul 25, 2016 · e0595bb · e0595bb
1 parent 85dcbf8
commit e0595bb
Show file tree

Hide file tree

Showing 4 changed files with 21 additions and 27 deletions.
diff --git a/src/renderers/dom/shared/ReactDOMDebugTool.js b/src/renderers/dom/shared/ReactDOMDebugTool.js
@@ -11,9 +11,9 @@
 
 'use strict';
 
+var ReactDebugTool = require('ReactDebugTool');
 var ReactDOMNullInputValuePropDevtool = require('ReactDOMNullInputValuePropDevtool');
 var ReactDOMUnknownPropertyDevtool = require('ReactDOMUnknownPropertyDevtool');
-var ReactDebugTool = require('ReactDebugTool');
 
 var warning = require('warning');
 
@@ -66,7 +66,9 @@ var ReactDOMDebugTool = {
   },
 };
 
-ReactDOMDebugTool.addDevtool(ReactDOMUnknownPropertyDevtool);
-ReactDOMDebugTool.addDevtool(ReactDOMNullInputValuePropDevtool);
+if (__DEV__) {
+  ReactDOMDebugTool.addDevtool(ReactDOMUnknownPropertyDevtool);
+  ReactDOMDebugTool.addDevtool(ReactDOMNullInputValuePropDevtool);
+}
 
 module.exports = ReactDOMDebugTool;
diff --git a/src/renderers/dom/shared/ReactDOMInstrumentation.js b/src/renderers/dom/shared/ReactDOMInstrumentation.js
@@ -11,11 +11,6 @@
 
 'use strict';
 
-var debugTool = null;
+var ReactDOMDebugTool = require('ReactDOMDebugTool');
 
-if (__DEV__) {
-  var ReactDOMDebugTool = require('ReactDOMDebugTool');
-  debugTool = ReactDOMDebugTool;
-}
-
-module.exports = {debugTool};
+module.exports = {debugTool: ReactDOMDebugTool};
diff --git a/src/renderers/shared/ReactDebugTool.js b/src/renderers/shared/ReactDebugTool.js
@@ -11,11 +11,11 @@
 
 'use strict';
 
-var ReactInvalidSetStateWarningDevTool = require('ReactInvalidSetStateWarningDevTool');
-var ReactHostOperationHistoryDevtool = require('ReactHostOperationHistoryDevtool');
-var ReactComponentTreeDevtool = require('ReactComponentTreeDevtool');
-var ReactChildrenMutationWarningDevtool = require('ReactChildrenMutationWarningDevtool');
 var ExecutionEnvironment = require('ExecutionEnvironment');
+var ReactChildrenMutationWarningDevtool = require('ReactChildrenMutationWarningDevtool');
+var ReactComponentTreeDevtool = require('ReactComponentTreeDevtool');
+var ReactHostOperationHistoryDevtool = require('ReactHostOperationHistoryDevtool');
+var ReactInvalidSetStateWarningDevTool = require('ReactInvalidSetStateWarningDevTool');
 
 var performanceNow = require('performanceNow');
 var warning = require('warning');
@@ -325,12 +325,14 @@ var ReactDebugTool = {
   },
 };
 
-ReactDebugTool.addDevtool(ReactInvalidSetStateWarningDevTool);
-ReactDebugTool.addDevtool(ReactComponentTreeDevtool);
-ReactDebugTool.addDevtool(ReactChildrenMutationWarningDevtool);
-var url = (ExecutionEnvironment.canUseDOM && window.location.href) || '';
-if ((/[?&]react_perf\b/).test(url)) {
-  ReactDebugTool.beginProfiling();
+if (__DEV__) {
+  ReactDebugTool.addDevtool(ReactInvalidSetStateWarningDevTool);
+  ReactDebugTool.addDevtool(ReactComponentTreeDevtool);
+  ReactDebugTool.addDevtool(ReactChildrenMutationWarningDevtool);
+  var url = (ExecutionEnvironment.canUseDOM && window.location.href) || '';
+  if ((/[?&]react_perf\b/).test(url)) {
+    ReactDebugTool.beginProfiling();
+  }
 }
 
 module.exports = ReactDebugTool;
diff --git a/src/renderers/shared/ReactInstrumentation.js b/src/renderers/shared/ReactInstrumentation.js
@@ -11,11 +11,6 @@
 
 'use strict';
 
-var debugTool = null;
+var ReactDebugTool = require('ReactDebugTool');
 
-if (__DEV__) {
-  var ReactDebugTool = require('ReactDebugTool');
-  debugTool = ReactDebugTool;
-}
-
-module.exports = {debugTool};
+module.exports = {debugTool: ReactDebugTool};