fix: avoid creating multiple promises

vitejs · Apr 3, 2023 · 462597b · 462597b
1 parent 62af5bc
commit 462597b
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/packages/vite/src/client/client.ts b/packages/vite/src/client/client.ts
@@ -328,6 +328,8 @@ async function waitForSuccessfulPing(
     return false
   }
 
+  let waitForWindowShowPromise: Promise<void> | undefined
+
   // eslint-disable-next-line no-constant-condition
   while (true) {
     if (document.visibilityState === 'visible') {
@@ -336,7 +338,12 @@ async function waitForSuccessfulPing(
       }
       await wait(ms)
     } else {
-      await Promise.race([wait(ms), waitForWindowShow()])
+      if (!waitForWindowShowPromise) {
+        waitForWindowShowPromise = waitForWindowShow().then(() => {
+          waitForWindowShowPromise = undefined
+        })
+      }
+      await Promise.race([wait(ms), waitForWindowShowPromise])
     }
   }
 }