Skip to content

Commit

Permalink
Dim console calls on additional Effect invocations due to `StrictMo…
Browse files Browse the repository at this point in the history
…de` (#29007)

DiffTrain build for [3ac551e](3ac551e)
  • Loading branch information
eps1lon committed May 22, 2024
1 parent ca580d7 commit b273ac4
Show file tree
Hide file tree
Showing 9 changed files with 33 additions and 9 deletions.
2 changes: 1 addition & 1 deletion compiled/facebook-www/REVISION
Original file line number Diff line number Diff line change
@@ -1 +1 @@
217b2ccf160d7594a926d276a75c7312b299be4d
3ac551e855f9bec3161da2fc8787958aa62113db
5 changes: 4 additions & 1 deletion compiled/facebook-www/ReactART-dev.classic.js
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ function _assertThisInitialized(self) {
return self;
}

var ReactVersion = '19.0.0-www-classic-27733b1d';
var ReactVersion = '19.0.0-www-classic-52a43e30';

var LegacyRoot = 0;
var ConcurrentRoot = 1;
Expand Down Expand Up @@ -24687,6 +24687,7 @@ function recursivelyTraverseAndDoubleInvokeEffectsInDEV(root, parentFiber, isInS

function doubleInvokeEffectsOnFiber(root, fiber) {
var shouldDoubleInvokePassiveEffects = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : true;
setIsStrictModeForDevtools(true);
disappearLayoutEffects(fiber);

if (shouldDoubleInvokePassiveEffects) {
Expand All @@ -24698,6 +24699,8 @@ function doubleInvokeEffectsOnFiber(root, fiber) {
if (shouldDoubleInvokePassiveEffects) {
reconnectPassiveEffects(root, fiber, NoLanes, null, false);
}

setIsStrictModeForDevtools(false);
}

function doubleInvokeEffectsInDEVIfNecessary(root, fiber, parentIsInStrictMode) {
Expand Down
5 changes: 4 additions & 1 deletion compiled/facebook-www/ReactART-dev.modern.js
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ function _assertThisInitialized(self) {
return self;
}

var ReactVersion = '19.0.0-www-modern-bddd4eac';
var ReactVersion = '19.0.0-www-modern-c5403897';

var LegacyRoot = 0;
var ConcurrentRoot = 1;
Expand Down Expand Up @@ -24041,6 +24041,7 @@ function recursivelyTraverseAndDoubleInvokeEffectsInDEV(root, parentFiber, isInS

function doubleInvokeEffectsOnFiber(root, fiber) {
var shouldDoubleInvokePassiveEffects = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : true;
setIsStrictModeForDevtools(true);
disappearLayoutEffects(fiber);

if (shouldDoubleInvokePassiveEffects) {
Expand All @@ -24052,6 +24053,8 @@ function doubleInvokeEffectsOnFiber(root, fiber) {
if (shouldDoubleInvokePassiveEffects) {
reconnectPassiveEffects(root, fiber, NoLanes, null, false);
}

setIsStrictModeForDevtools(false);
}

function doubleInvokeEffectsInDEVIfNecessary(root, fiber, parentIsInStrictMode) {
Expand Down
5 changes: 4 additions & 1 deletion compiled/facebook-www/ReactDOM-dev.classic.js
Original file line number Diff line number Diff line change
Expand Up @@ -29418,6 +29418,7 @@ function recursivelyTraverseAndDoubleInvokeEffectsInDEV(root, parentFiber, isInS

function doubleInvokeEffectsOnFiber(root, fiber) {
var shouldDoubleInvokePassiveEffects = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : true;
setIsStrictModeForDevtools(true);
disappearLayoutEffects(fiber);

if (shouldDoubleInvokePassiveEffects) {
Expand All @@ -29429,6 +29430,8 @@ function doubleInvokeEffectsOnFiber(root, fiber) {
if (shouldDoubleInvokePassiveEffects) {
reconnectPassiveEffects(root, fiber, NoLanes, null, false);
}

setIsStrictModeForDevtools(false);
}

function doubleInvokeEffectsInDEVIfNecessary(root, fiber, parentIsInStrictMode) {
Expand Down Expand Up @@ -30852,7 +30855,7 @@ identifierPrefix, onUncaughtError, onCaughtError, onRecoverableError, transition
return root;
}

var ReactVersion = '19.0.0-www-classic-568ec550';
var ReactVersion = '19.0.0-www-classic-48001749';

function createPortal$1(children, containerInfo, // TODO: figure out the API for cross-renderer implementation.
implementation) {
Expand Down
5 changes: 4 additions & 1 deletion compiled/facebook-www/ReactDOM-dev.modern.js
Original file line number Diff line number Diff line change
Expand Up @@ -28669,6 +28669,7 @@ function recursivelyTraverseAndDoubleInvokeEffectsInDEV(root, parentFiber, isInS

function doubleInvokeEffectsOnFiber(root, fiber) {
var shouldDoubleInvokePassiveEffects = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : true;
setIsStrictModeForDevtools(true);
disappearLayoutEffects(fiber);

if (shouldDoubleInvokePassiveEffects) {
Expand All @@ -28680,6 +28681,8 @@ function doubleInvokeEffectsOnFiber(root, fiber) {
if (shouldDoubleInvokePassiveEffects) {
reconnectPassiveEffects(root, fiber, NoLanes, null, false);
}

setIsStrictModeForDevtools(false);
}

function doubleInvokeEffectsInDEVIfNecessary(root, fiber, parentIsInStrictMode) {
Expand Down Expand Up @@ -30027,7 +30030,7 @@ identifierPrefix, onUncaughtError, onCaughtError, onRecoverableError, transition
return root;
}

var ReactVersion = '19.0.0-www-modern-1d14d92b';
var ReactVersion = '19.0.0-www-modern-899aca39';

function createPortal$1(children, containerInfo, // TODO: figure out the API for cross-renderer implementation.
implementation) {
Expand Down
5 changes: 4 additions & 1 deletion compiled/facebook-www/ReactDOMTesting-dev.classic.js
Original file line number Diff line number Diff line change
Expand Up @@ -29984,6 +29984,7 @@ function recursivelyTraverseAndDoubleInvokeEffectsInDEV(root, parentFiber, isInS

function doubleInvokeEffectsOnFiber(root, fiber) {
var shouldDoubleInvokePassiveEffects = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : true;
setIsStrictModeForDevtools(true);
disappearLayoutEffects(fiber);

if (shouldDoubleInvokePassiveEffects) {
Expand All @@ -29995,6 +29996,8 @@ function doubleInvokeEffectsOnFiber(root, fiber) {
if (shouldDoubleInvokePassiveEffects) {
reconnectPassiveEffects(root, fiber, NoLanes, null, false);
}

setIsStrictModeForDevtools(false);
}

function doubleInvokeEffectsInDEVIfNecessary(root, fiber, parentIsInStrictMode) {
Expand Down Expand Up @@ -31418,7 +31421,7 @@ identifierPrefix, onUncaughtError, onCaughtError, onRecoverableError, transition
return root;
}

var ReactVersion = '19.0.0-www-classic-96e3c07d';
var ReactVersion = '19.0.0-www-classic-5c88d62e';

function createPortal$1(children, containerInfo, // TODO: figure out the API for cross-renderer implementation.
implementation) {
Expand Down
5 changes: 4 additions & 1 deletion compiled/facebook-www/ReactDOMTesting-dev.modern.js
Original file line number Diff line number Diff line change
Expand Up @@ -29235,6 +29235,7 @@ function recursivelyTraverseAndDoubleInvokeEffectsInDEV(root, parentFiber, isInS

function doubleInvokeEffectsOnFiber(root, fiber) {
var shouldDoubleInvokePassiveEffects = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : true;
setIsStrictModeForDevtools(true);
disappearLayoutEffects(fiber);

if (shouldDoubleInvokePassiveEffects) {
Expand All @@ -29246,6 +29247,8 @@ function doubleInvokeEffectsOnFiber(root, fiber) {
if (shouldDoubleInvokePassiveEffects) {
reconnectPassiveEffects(root, fiber, NoLanes, null, false);
}

setIsStrictModeForDevtools(false);
}

function doubleInvokeEffectsInDEVIfNecessary(root, fiber, parentIsInStrictMode) {
Expand Down Expand Up @@ -30593,7 +30596,7 @@ identifierPrefix, onUncaughtError, onCaughtError, onRecoverableError, transition
return root;
}

var ReactVersion = '19.0.0-www-modern-8378a978';
var ReactVersion = '19.0.0-www-modern-8a4086e1';

function createPortal$1(children, containerInfo, // TODO: figure out the API for cross-renderer implementation.
implementation) {
Expand Down
5 changes: 4 additions & 1 deletion compiled/facebook-www/ReactReconciler-dev.classic.js
Original file line number Diff line number Diff line change
Expand Up @@ -27208,6 +27208,7 @@ function recursivelyTraverseAndDoubleInvokeEffectsInDEV(root, parentFiber, isInS

function doubleInvokeEffectsOnFiber(root, fiber) {
var shouldDoubleInvokePassiveEffects = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : true;
setIsStrictModeForDevtools(true);
disappearLayoutEffects(fiber);

if (shouldDoubleInvokePassiveEffects) {
Expand All @@ -27219,6 +27220,8 @@ function doubleInvokeEffectsOnFiber(root, fiber) {
if (shouldDoubleInvokePassiveEffects) {
reconnectPassiveEffects(root, fiber, NoLanes, null, false);
}

setIsStrictModeForDevtools(false);
}

function doubleInvokeEffectsInDEVIfNecessary(root, fiber, parentIsInStrictMode) {
Expand Down Expand Up @@ -28650,7 +28653,7 @@ identifierPrefix, onUncaughtError, onCaughtError, onRecoverableError, transition
return root;
}

var ReactVersion = '19.0.0-www-classic-d0612afb';
var ReactVersion = '19.0.0-www-classic-e2e5cb48';

/*
* The `'' + value` pattern (used in perf-sensitive code) throws for Symbol
Expand Down
5 changes: 4 additions & 1 deletion compiled/facebook-www/ReactReconciler-dev.modern.js
Original file line number Diff line number Diff line change
Expand Up @@ -26550,6 +26550,7 @@ function recursivelyTraverseAndDoubleInvokeEffectsInDEV(root, parentFiber, isInS

function doubleInvokeEffectsOnFiber(root, fiber) {
var shouldDoubleInvokePassiveEffects = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : true;
setIsStrictModeForDevtools(true);
disappearLayoutEffects(fiber);

if (shouldDoubleInvokePassiveEffects) {
Expand All @@ -26561,6 +26562,8 @@ function doubleInvokeEffectsOnFiber(root, fiber) {
if (shouldDoubleInvokePassiveEffects) {
reconnectPassiveEffects(root, fiber, NoLanes, null, false);
}

setIsStrictModeForDevtools(false);
}

function doubleInvokeEffectsInDEVIfNecessary(root, fiber, parentIsInStrictMode) {
Expand Down Expand Up @@ -27916,7 +27919,7 @@ identifierPrefix, onUncaughtError, onCaughtError, onRecoverableError, transition
return root;
}

var ReactVersion = '19.0.0-www-modern-80961c3b';
var ReactVersion = '19.0.0-www-modern-a0174ad9';

/*
* The `'' + value` pattern (used in perf-sensitive code) throws for Symbol
Expand Down

0 comments on commit b273ac4

Please sign in to comment.