Skip to content

Commit ceee524

Browse files
authored
Remove unnecessary clearContainer call (#22979)
This was added when we added error recovery for hydration errors. However, when the fix up pass happens later on, it'll still call clearContainer in the commit phase. So this call is unnecessary.
1 parent aa8f2bd commit ceee524

File tree

2 files changed

+0
-4
lines changed

2 files changed

+0
-4
lines changed

packages/react-reconciler/src/ReactFiberWorkLoop.new.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,6 @@ import {
8686
cancelTimeout,
8787
noTimeout,
8888
afterActiveInstanceBlur,
89-
clearContainer,
9089
getCurrentEventPriority,
9190
supportsMicrotasks,
9291
errorHydratingContainer,
@@ -885,7 +884,6 @@ function recoverFromConcurrentError(root, errorRetryLanes) {
885884
if (__DEV__) {
886885
errorHydratingContainer(root.containerInfo);
887886
}
888-
clearContainer(root.containerInfo);
889887
}
890888

891889
let exitStatus;

packages/react-reconciler/src/ReactFiberWorkLoop.old.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,6 @@ import {
8686
cancelTimeout,
8787
noTimeout,
8888
afterActiveInstanceBlur,
89-
clearContainer,
9089
getCurrentEventPriority,
9190
supportsMicrotasks,
9291
errorHydratingContainer,
@@ -885,7 +884,6 @@ function recoverFromConcurrentError(root, errorRetryLanes) {
885884
if (__DEV__) {
886885
errorHydratingContainer(root.containerInfo);
887886
}
888-
clearContainer(root.containerInfo);
889887
}
890888

891889
let exitStatus;

0 commit comments

Comments
 (0)