patch(playwright): crxZone that provides api name

Required for tracing
ruifigueira · Dec 11, 2024 · f3d6582 · f3d6582
1 parent 4998414
commit f3d6582
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/playwright/packages/playwright-core/src/client/channelOwner.ts b/playwright/packages/playwright-core/src/client/channelOwner.ts
@@ -174,8 +174,10 @@ export abstract class ChannelOwner<T extends channels.Channel = channels.Channel
     if (apiZone)
       return await func(apiZone);
 
+    const crxZone = zones.zoneData<{ apiName: string }>('crxZone');
+
     const stackTrace = captureLibraryStackTrace();
-    let apiName: string | undefined = stackTrace.apiName;
+    let apiName: string | undefined = crxZone?.apiName ?? stackTrace.apiName;
     const frames: channels.StackFrame[] = stackTrace.frames;
 
     if (isInternal === undefined)

diff --git a/playwright/packages/playwright-core/src/utils/zones.ts b/playwright/packages/playwright-core/src/utils/zones.ts
@@ -16,7 +16,7 @@
 
 import { AsyncLocalStorage } from 'async_hooks';
 
-export type ZoneType = 'apiZone' | 'expectZone' | 'stepZone';
+export type ZoneType = 'crxZone' | 'apiZone' | 'expectZone' | 'stepZone';
 
 class ZoneManager {
   private readonly _asyncLocalStorage = new AsyncLocalStorage<Zone|undefined>();