chore(bidi): fix intercepts for context.route()

Author: hbenl

packages/playwright-core/src/server/bidi/bidiBrowser.ts

@@ -191,6 +191,7 @@ export class BidiBrowserContext extends BrowserContext {
   declare readonly _browser: BidiBrowser;
   private _originToPermissions = new Map<string, string[]>();
   private _initScriptIds = new Map<InitScript, string>();
+  private _interceptIdPromise: Promise<bidi.Network.Intercept> | undefined;
 
   constructor(browser: BidiBrowser, browserContextId: string | undefined, options: types.BrowserContextOptions) {
     super(browser, options, browserContextId);
@@ -373,6 +374,19 @@ export class BidiBrowserContext extends BrowserContext {
   }
 
   async doUpdateRequestInterception(): Promise<void> {
+    if (this.requestInterceptors.length > 0 && !this._interceptIdPromise) {
+      this._interceptIdPromise = this._browser._browserSession.send('network.addIntercept', {
+        phases: [bidi.Network.InterceptPhase.BeforeRequestSent],
+        urlPatterns: [{ type: 'pattern' }],
+      }).then(({ intercept }) => intercept);
+    }
+    if (this.requestInterceptors.length === 0 && this._interceptIdPromise) {
+      const interceptIdPromise = this._interceptIdPromise;
+      this._interceptIdPromise = undefined;
+      await this._browser._browserSession.send('network.removeIntercept', {
+        intercept: await interceptIdPromise
+      });
+    }
   }
 
   override async doUpdateDefaultViewport() {

packages/playwright-core/src/server/bidi/bidiPage.ts

@@ -95,7 +95,6 @@ export class BidiPage implements PageDelegate {
     this._onFrameAttached(this._session.sessionId, null);
     await Promise.all([
       this.updateHttpCredentials(),
-      this.updateRequestInterception(),
       // If the page is created by the Playwright client's call, some initialization
       // may be pending. Wait for it to complete before reporting the page as new.
     ]);
@@ -345,7 +344,7 @@ export class BidiPage implements PageDelegate {
   }
 
   async updateRequestInterception(): Promise<void> {
-    await this._networkManager.setRequestInterception(this._page.needsRequestInterception());
+    await this._networkManager.setRequestInterception(this._page.requestInterceptors.length > 0);
   }
 
   async updateOffline() {