fix(puppet): workers not tracking devtools calls

Author: blakebyrnes

puppet-chrome/lib/BrowserContext.ts

@@ -113,11 +113,11 @@ export class BrowserContext
     }
   }
 
-  onWorkerAttached(cdpSession: CDPSession, worker: Worker, pageTargetId: string) {
+  onWorkerAttached(cdpSession: CDPSession, workerTargetId: string, pageTargetId: string) {
     this.subscribeToDevtoolsMessages(cdpSession, {
       sessionType: 'worker' as const,
       pageTargetId,
-      workerTargetId: worker.id,
+      workerTargetId,
     });
   }
 

puppet-chrome/lib/Page.ts

@@ -288,14 +288,14 @@ export class Page extends TypedEventEmitter<IPuppetPageEvents> implements IPuppe
     const cdpSession = this.cdpSession.connection.getSession(sessionId);
 
     if (targetInfo.type === 'service_worker' || targetInfo.type === 'worker') {
+      this.browserContext.onWorkerAttached(cdpSession, targetInfo.targetId, this.targetId);
       const worker = new Worker(
         this.browserContext,
         this.networkManager,
         cdpSession,
         this.logger,
         targetInfo,
       );
-      this.browserContext.onWorkerAttached(cdpSession, worker, this.targetId);
       const targetId = targetInfo.targetId;
       this.workersById.set(targetId, worker);