Enable MessageLoop implementation by default

facebook · Aug 15, 2019 · d61c6cc · d61c6cc
1 parent 868d02d
commit d61c6cc
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 5 deletions.
diff --git a/packages/scheduler/src/SchedulerFeatureFlags.js b/packages/scheduler/src/SchedulerFeatureFlags.js
@@ -10,4 +10,4 @@ export const enableSchedulerDebugging = false;
 export const enableIsInputPending = false;
 export const requestIdleCallbackBeforeFirstFrame = false;
 export const requestTimerEventBeforeFirstFrame = false;
-export const enableMessageLoopImplementation = false;
+export const enableMessageLoopImplementation = true;
diff --git a/...eduler/src/__tests__/SchedulerDOM-test.js → ..._tests__/SchedulerRAFOld-test.internal.js b/...eduler/src/__tests__/SchedulerDOM-test.js → ..._tests__/SchedulerRAFOld-test.internal.js
@@ -16,7 +16,9 @@ type FrameTimeoutConfigType = {
   timePastFrameDeadline: ?number,
 };
 
-describe('SchedulerDOM', () => {
+// Note: this is testing the implementation that we turned off.
+// enableMessageLoopImplementation is true on master.
+describe('SchedulerRAFOld', () => {
   let rAFCallbacks = [];
   let postMessageCallback;
   let postMessageEvents = [];
@@ -97,6 +99,7 @@ describe('SchedulerDOM', () => {
       ),
     );
 
+    require('scheduler/src/SchedulerFeatureFlags').enableMessageLoopImplementation = false;
     Scheduler = require('scheduler');
   });
 

diff --git a/packages/scheduler/src/forks/SchedulerFeatureFlags.www.js b/packages/scheduler/src/forks/SchedulerFeatureFlags.www.js
@@ -9,7 +9,8 @@
 export const {
   enableIsInputPending,
   enableSchedulerDebugging,
-  requestIdleCallbackBeforeFirstFrame,
-  requestTimerEventBeforeFirstFrame,
-  enableMessageLoopImplementation,
 } = require('SchedulerFeatureFlags');
+
+export const requestIdleCallbackBeforeFirstFrame = false;
+export const requestTimerEventBeforeFirstFrame = false;
+export const enableMessageLoopImplementation = true;