feat: support pageLoadStrategy in readState check

lib/mixins/navigate.js

@@ -32,6 +32,21 @@ function cancelPageLoad () {
   }
 }
 
+/**
+ * Return if current readState can be handles as page load completes
+ * for the given page load strategy,
+ * @param {string} pageLoadStrategy
+ * @param {string} readyState
+ * @returns string
+ */
+function completeToLoadPage (pageLoadStrategy, readyState) {
+  if (pageLoadStrategy === 'eager') {
+    // this could include 'interactive', or 'complete'
+    return readyState !== 'loading';
+  }
+  return readyState === 'complete';
+}
+
 /**
  * @this {import('../remote-debugger').RemoteDebugger}
  * @param {timing.Timer|null|undefined} startPageLoadTimer
@@ -114,7 +129,7 @@ async function checkPageIsReady (timeoutMs) {
     const readyState = await B.resolve(this.execute(readyCmd, true))
       .timeout(timeoutMs ?? this.pageReadyTimeout);
     log.debug(`Document readyState is '${readyState}'`);
-    return readyState === 'complete';
+    return completeToLoadPage(this.pageLoadStrategy, readyState);
   } catch (err) {
     if (!(err instanceof B.TimeoutError)) {
       throw err;
@@ -187,6 +202,11 @@ async function navToUrl (url) {
     // and start sending `document.readyState` requests until we either succeed or
     // another event/timeout happens
     onTargetProvisioned = async () => {
+      if (this.pageLoadStrategy === 'none') {
+        log.debug("none page load strategy");
+        return
+      };
+
       while (isPageLoading) {
         const pageReadyCheckStart = new timing.Timer().start();
         try {

lib/remote-debugger.js

@@ -39,12 +39,16 @@ class RemoteDebugger extends EventEmitter {
   navigationDelay;
   /** @type {import('./rpc/rpc-client').default?} */
   rpcClient;
+  /** @type {string} */
+  pageLoadStrategy;
 
   // events
   /** @type {string} */
   static EVENT_PAGE_CHANGE;
   /** @type {string} */
   static EVENT_DISCONNECT;
+  /** @type {string} */
+  static EVENT_FRAMES_DETACHED;
 
   // methods
   /** @type {() => Promise<void>} */
@@ -126,6 +130,7 @@ class RemoteDebugger extends EventEmitter {
       webInspectorMaxFrameLength,
       socketChunkSize,
       fullPageInitialization,
+      pageLoadStrategy = 'normal'
     } = opts;
 
     this.bundleId = bundleId;
@@ -155,6 +160,8 @@ class RemoteDebugger extends EventEmitter {
 
     this.fullPageInitialization = fullPageInitialization;
 
+    this.pageLoadStrategy = pageLoadStrategy;
+
     this._lock = new AsyncLock();
   }