chore: update TODOs (#12765)

Author: OrKoN

packages/puppeteer-core/src/bidi/BidiOverCdp.ts

@@ -56,9 +56,8 @@ export async function connectBidiOverCdp(
   const bidiServer = await BidiMapper.BidiServer.createAndStart(
     transportBiDi,
     cdpConnectionAdapter,
-    // TODO: most likely need a little bit of refactoring
     cdpConnectionAdapter.browserClient(),
-    '',
+    /* selfTargetId= */ '',
     {
       // Override Mapper's `unhandledPromptBehavior` default value of `dismiss` to
       // `ignore`, so that user can handle the prompt instead of just closing it.

packages/puppeteer-core/src/bidi/Browser.ts

@@ -48,7 +48,6 @@ export interface BidiBrowserOptions {
 export class BidiBrowser extends Browser {
   readonly protocol = 'webDriverBiDi';
 
-  // TODO: Update generator to include fully module
   static readonly subscribeModules: [string, ...string[]] = [
     'browsingContext',
     'network',

packages/puppeteer-core/src/bidi/BrowserConnector.ts

@@ -109,7 +109,6 @@ async function getBiDiConnection(
     );
   }
 
-  // TODO: use other options too.
   const bidiOverCdpConnection = await BiDi.connectBidiOverCdp(cdpConnection, {
     acceptInsecureCerts: ignoreHTTPSErrors,
   });

packages/puppeteer-core/src/bidi/HTTPResponse.ts

@@ -105,7 +105,6 @@ export class BidiHTTPResponse extends HTTPResponse {
   }
 
   override timing(): Protocol.Network.ResourceTiming | null {
-    // TODO: File and issue with BiDi spec
     throw new UnsupportedOperation();
   }
 

packages/puppeteer-core/src/bidi/core/Session.ts

@@ -17,9 +17,6 @@ import {DisposableStack, disposeSymbol} from '../../util/disposable.js';
 import {Browser} from './Browser.js';
 import type {BidiEvents, Commands, Connection} from './Connection.js';
 
-// TODO: Once Chrome supports session.status properly, uncomment this block.
-// const MAX_RETRIES = 5;
-
 /**
  * @internal
  */
@@ -31,26 +28,6 @@ export class Session
     connection: Connection,
     capabilities: Bidi.Session.CapabilitiesRequest
   ): Promise<Session> {
-    // Wait until the session is ready.
-    //
-    // TODO: Once Chrome supports session.status properly, uncomment this block
-    // and remove `getBiDiConnection` in BrowserConnector.
-
-    // let status = {message: '', ready: false};
-    // for (let i = 0; i < MAX_RETRIES; ++i) {
-    //   status = (await connection.send('session.status', {})).result;
-    //   if (status.ready) {
-    //     break;
-    //   }
-    //   // Backoff a little bit each time.
-    //   await new Promise(resolve => {
-    //     return setTimeout(resolve, (1 << i) * 100);
-    //   });
-    // }
-    // if (!status.ready) {
-    //   throw new Error(status.message);
-    // }
-
     const {result} = await connection.send('session.new', {
       capabilities,
     });