[Feature]: pass the locator as a parameter to the handler of addLocatorHandler

[regseb]

🚀 Feature Request

With Page.addLocatorHandler(), pass the locator as a parameter to the handler function.

Example

• Currently:

  await page.addLocatorHandler(
    page.getByRole('button', { name: 'Accept all cookies' }),
    () => page.getByRole('button', { name: 'Accept all cookies' }).click());

• With this feature:

  await page.addLocatorHandler(
    page.getByRole('button', { name: 'Accept all cookies' }),
    (locator) => locator.click());

Motivation

• Do not write the same selector twice.
• Use a single generic handler to close several dialog.

[regseb]

Suggested modification to packages/playwright-core/src/client/page.ts:

   async addLocatorHandler(locator: Locator, handler: Function): Promise<void> {
     if (locator._frame !== this._mainFrame)
       throw new Error(`Locator must belong to the main frame of this page`);
     const { uid } = await this._channel.registerLocatorHandler({ selector: locator._selector });
-    this._locatorHandlers.set(uid, handler);
+    this._locatorHandlers.set(uid, { locator, handler });
   }

   private async _onLocatorHandlerTriggered(uid: number) {
     try {
-      const handler = this._locatorHandlers.get(uid);
-      await handler?.();
+      const { locator, handler } = this._locatorHandlers.get(uid);
+      await handler?.(locator);
     } finally {
       this._wrapApiCall(() => this._channel.resolveLocatorHandlerNoReply({ uid }), true).catch(() => {});
     }
   }