Always use page.waitForNetworkIdle() to wait initial rendering

Author: yhatt

src/converter.ts

@@ -726,22 +726,17 @@ export class Converter {
         this.trackFailedLocalFileAccess(page)
 
       const render = async () => {
+        const waitUntil = 'domcontentloaded' as const
+
         if (uri) {
-          await page.goto(uri, {
-            waitUntil: ['domcontentloaded', 'networkidle0'],
-          })
+          await page.goto(uri, { waitUntil })
         } else {
-          await page.goto('data:text/html,', {
-            waitUntil: ['domcontentloaded', 'networkidle0'],
-          })
-
-          // https://github.com/marp-team/marp-cli/issues/682 -- Use page.waitForNetworkIdle() instead of `waitUntil: 'networkidle0'`
-          await page.setContent(baseFile.buffer!.toString(), {
-            waitUntil: 'domcontentloaded',
-          })
-          await page.waitForNetworkIdle()
+          await page.goto('data:text/html,', { waitUntil })
+          await page.setContent(baseFile.buffer!.toString(), { waitUntil })
         }
 
+        await page.waitForNetworkIdle()
+
         // Wait for next frame (In parallel rendering, it may be needed to wait for the first rendering)
         await page.evaluate(async () => {
           /* c8 ignore start */