Rename onComplete -> onCompleteAll

facebook · Mar 23, 2021 · b28626f · b28626f
1 parent 0509b57
commit b28626f
Show file tree

Hide file tree

Showing 6 changed files with 15 additions and 15 deletions.
diff --git a/packages/react-dom/src/__tests__/ReactDOMFizzServerBrowser-test.js b/packages/react-dom/src/__tests__/ReactDOMFizzServerBrowser-test.js
@@ -77,7 +77,7 @@ describe('ReactDOMFizzServer', () => {
         </Suspense>
       </div>,
       {
-        onComplete() {
+        onCompleteAll() {
           isComplete = true;
         },
       },

diff --git a/packages/react-dom/src/__tests__/ReactDOMFizzServerNode-test.js b/packages/react-dom/src/__tests__/ReactDOMFizzServerNode-test.js
@@ -107,7 +107,7 @@ describe('ReactDOMFizzServer', () => {
       </div>,
       writable,
       {
-        onComplete() {
+        onCompleteAll() {
           isComplete = true;
         },
       },

diff --git a/packages/react-dom/src/server/ReactDOMFizzServerBrowser.js b/packages/react-dom/src/server/ReactDOMFizzServerBrowser.js
@@ -23,7 +23,7 @@ type Options = {
   progressiveChunkSize?: number,
   signal?: AbortSignal,
   onReadyToStream?: () => void,
-  onComplete?: () => void,
+  onCompleteAll?: () => void,
   onError?: (error: mixed) => void,
 };
 
@@ -48,7 +48,7 @@ function renderToReadableStream(
         createResponseState(options ? options.identifierPrefix : undefined),
         options ? options.progressiveChunkSize : undefined,
         options ? options.onError : undefined,
-        options ? options.onComplete : undefined,
+        options ? options.onCompleteAll : undefined,
         options ? options.onReadyToStream : undefined,
       );
       startWork(request);

diff --git a/packages/react-dom/src/server/ReactDOMFizzServerNode.js b/packages/react-dom/src/server/ReactDOMFizzServerNode.js
@@ -27,7 +27,7 @@ type Options = {
   identifierPrefix?: string,
   progressiveChunkSize?: number,
   onReadyToStream?: () => void,
-  onComplete?: () => void,
+  onCompleteAll?: () => void,
   onError?: (error: mixed) => void,
 };
 
@@ -48,7 +48,7 @@ function pipeToNodeWritable(
     createResponseState(options ? options.identifierPrefix : undefined),
     options ? options.progressiveChunkSize : undefined,
     options ? options.onError : undefined,
-    options ? options.onComplete : undefined,
+    options ? options.onCompleteAll : undefined,
     options ? options.onReadyToStream : undefined,
   );
   let hasStartedFlowing = false;

diff --git a/packages/react-noop-renderer/src/ReactNoopServer.js b/packages/react-noop-renderer/src/ReactNoopServer.js
@@ -218,7 +218,7 @@ const ReactNoopServer = ReactFizzServer({
 type Options = {
   progressiveChunkSize?: number,
   onReadyToStream?: () => void,
-  onComplete?: () => void,
+  onCompleteAll?: () => void,
   onError?: (error: mixed) => void,
 };
 
@@ -238,7 +238,7 @@ function render(children: React$Element<any>, options?: Options): Destination {
     null,
     options ? options.progressiveChunkSize : undefined,
     options ? options.onError : undefined,
-    options ? options.onComplete : undefined,
+    options ? options.onCompleteAll : undefined,
     options ? options.onReadyToStream : undefined,
   );
   ReactNoopServer.startWork(request);

diff --git a/packages/react-server/src/ReactFizzServer.js b/packages/react-server/src/ReactFizzServer.js
@@ -112,9 +112,9 @@ type Request = {
   partialBoundaries: Array<SuspenseBoundary>, // Partially completed boundaries that can flush its segments early.
   // onError is called when an error happens anywhere in the tree. It might recover.
   onError: (error: mixed) => void,
-  // onComplete is called when all pending work is done but it may not have flushed yet.
+  // onCompleteAll is called when all pending work is done but it may not have flushed yet.
   // This is a good time to start writing if you want only HTML and no intermediate steps.
-  onComplete: () => void,
+  onCompleteAll: () => void,
   // onReadyToStream is called when there is at least a root fallback ready to show.
   // Typically you don't need this callback because it's best practice to always have a
   // root fallback ready so there's no need to wait.
@@ -144,7 +144,7 @@ export function createRequest(
   responseState: ResponseState,
   progressiveChunkSize: number = DEFAULT_PROGRESSIVE_CHUNK_SIZE,
   onError: (error: mixed) => void = noop,
-  onComplete: () => void = noop,
+  onCompleteAll: () => void = noop,
   onReadyToStream: () => void = noop,
 ): Request {
   const pingedWork = [];
@@ -164,7 +164,7 @@ export function createRequest(
     completedBoundaries: [],
     partialBoundaries: [],
     onError,
-    onComplete,
+    onCompleteAll,
     onReadyToStream,
   };
   // This segment represents the root fallback.
@@ -429,7 +429,7 @@ function erroredWork(
 
   request.allPendingWork--;
   if (request.allPendingWork === 0) {
-    request.onComplete();
+    request.onCompleteAll();
   }
 }
 
@@ -476,7 +476,7 @@ function abortWork(suspendedWork: SuspendedWork): void {
     }
 
     if (request.allPendingWork === 0) {
-      request.onComplete();
+      request.onCompleteAll();
     }
   }
 }
@@ -537,7 +537,7 @@ function finishedWork(
   if (request.allPendingWork === 0) {
     // This needs to be called at the very end so that we can synchronously write the result
     // in the callback if needed.
-    request.onComplete();
+    request.onCompleteAll();
   }
 }