Only affect OSS for now

Author: eps1lon

packages/react-reconciler/src/__tests__/ReactIsomorphicAct-test.js

@@ -50,9 +50,16 @@ describe('isomorphic act()', () => {
     return text;
   }
 
-  // @gate !__DEV__
-  it('is not exported in production builds', () => {
-    expect(React).not.toHaveProperty('act');
+  it('behavior in production', () => {
+    if (!__DEV__) {
+      if (gate('fb')) {
+        expect(() => act(() => {})).toThrow(
+          'act(...) is not supported in production builds of React',
+        );
+      } else {
+        expect(React).not.toHaveProperty('act');
+      }
+    }
   });
 
   // @gate __DEV__

packages/react-reconciler/src/__tests__/ReactOwnerStacks-test.js

@@ -31,12 +31,13 @@ describe('ReactOwnerStacks', () => {
     );
   }
 
-  it('is not exported in production buids', () => {
-    // awkward to gate because FB builds will have it undefined if disabled
-    // whereas OSS builds will have it stripped if disabled because we only have
-    // it on in experimental which has a separate entrypoint
+  it('behavior in production', () => {
     if (!__DEV__) {
-      expect(React).not.toHaveProperty('captureOwnerStack');
+      if (gate('fb')) {
+        expect(React).toHaveProperty('captureOwnerStack', undefined);
+      } else {
+        expect(React).not.toHaveProperty('captureOwnerStack');
+      }
     }
   });
 

packages/react/index.fb.development.js

@@ -7,9 +7,12 @@
  * @flow
  */
 
+import {enableOwnerStacks} from 'shared/ReactFeatureFlags';
+import {captureOwnerStack as captureOwnerStackImpl} from './src/ReactClient';
 export {
   __CLIENT_INTERNALS_DO_NOT_USE_OR_WARN_USERS_THEY_CANNOT_UPGRADE,
   __COMPILER_RUNTIME,
+  act,
   cache,
   Children,
   cloneElement,
@@ -66,3 +69,11 @@ export {useMemoCache as unstable_useMemoCache} from './src/ReactHooks';
 // export to match the name of the OSS function typically exported from
 // react/compiler-runtime
 export {useMemoCache as c} from './src/ReactHooks';
+
+// Only export captureOwnerStack in development.
+let captureOwnerStack: ?() => null | string;
+if (__DEV__ && enableOwnerStacks) {
+  captureOwnerStack = captureOwnerStackImpl;
+}
+
+export {captureOwnerStack};

packages/react/index.fb.js

@@ -78,6 +78,7 @@ function getTestFlags() {
       classic: releaseChannel === 'classic',
       source: !process.env.IS_BUILD,
       www,
+      fb: www || xplat,
 
       // These aren't flags, just a useful aliases for tests.
       enableActivity: releaseChannel === 'experimental' || www || xplat,