From 73418954f34259f71eac7db5610793c6f13ddfbd Mon Sep 17 00:00:00 2001 From: Sebastian Markbage Date: Tue, 12 Nov 2024 21:30:25 -0500 Subject: [PATCH] Use completedRenderEndTime as the start of the commit phase if it's an immediate commit We don't log a phase like "Throttled" or "Suspended" for this case so it can leave a tiny gap otherwise. --- packages/react-reconciler/src/ReactFiberWorkLoop.js | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/packages/react-reconciler/src/ReactFiberWorkLoop.js b/packages/react-reconciler/src/ReactFiberWorkLoop.js index bd30492fc0d0e..bffedf498bc28 100644 --- a/packages/react-reconciler/src/ReactFiberWorkLoop.js +++ b/packages/react-reconciler/src/ReactFiberWorkLoop.js @@ -3250,7 +3250,12 @@ function commitRootImpl( if (enableProfilerTimer && enableComponentPerformanceTrack) { recordCommitEndTime(); - logCommitPhase(commitStartTime, commitEndTime); + logCommitPhase( + suspendedCommitReason === IMMEDIATE_COMMIT + ? completedRenderEndTime + : commitStartTime, + commitEndTime, + ); } const rootDidHavePassiveEffects = rootDoesHavePassiveEffects;