test(webkit): no pageerror for a caught fetch during pending navigation (#44090)

Author: yury-s

tests/page/page-event-pageerror.spec.ts

@@ -16,6 +16,7 @@
  */
 
 import { test as it, expect } from './pageTest';
+import type { ServerResponse } from 'http';
 
 it('ER-001 should fire', {
   annotation: { type: 'issue', description: 'https://github.com/microsoft/playwright/issues/43450' },
@@ -284,3 +285,31 @@ it('ER-013 should fire illegal character error', {
   else
     expect(error.message).toContain('illegal character');
 });
+
+it('ER-022 should not fire for a caught fetch rejected by a pending navigation', {
+  annotation: { type: 'issue', description: 'https://github.com/microsoft/playwright/issues/44015' },
+}, async ({ page, server }) => {
+  await page.goto(server.EMPTY_PAGE);
+  const errors: string[] = [];
+  page.on('pageerror', error => errors.push(error.message));
+  const waitResponse = new Promise<ServerResponse>(resolve => server.setRoute('/wait', (req, res) => resolve(res)));
+  const replacementResponse = new Promise<ServerResponse>(resolve => server.setRoute('/replacement', (req, res) => resolve(res)));
+  const fetchDone = page.waitForEvent('console', message => message.text() === 'fetch done');
+  // Chromium does not run evaluate during a pending navigation, so the page starts the second fetch.
+  await page.evaluate(() => {
+    void fetch('/wait').catch(() => {}).then(() => fetch('/empty.html')).catch(() => {}).then(() => console.log('fetch done'));
+  });
+  const navigation = page.goto(server.PREFIX + '/replacement');
+  const response = await replacementResponse;
+  // WebKit rejects the second fetch, because the document is about to go away.
+  (await waitResponse).end();
+  await fetchDone;
+  response.end('replacement');
+  await navigation;
+  const [error] = await Promise.all([
+    page.waitForEvent('pageerror'),
+    page.evaluate(() => setTimeout(() => { throw new Error('genuine error'); }, 0)),
+  ]);
+  expect(error.message).toBe('genuine error');
+  expect(errors).toEqual(['genuine error']);
+});

tests/registry.md

@@ -2064,6 +2064,7 @@ subsystem, add a row to the Subsystems table. Then update this file:
 | `ER-019` | `library/page-event-crash.spec.ts` | should be able to close context when page crashes |
 | `ER-020` | `library/page-event-crash.spec.ts` | should be able to close page after crash |
 | `ER-021` | `library/page-event-crash.spec.ts` | should reject in-flight worker.evaluate when page crashes |
+| `ER-022` | `page/page-event-pageerror.spec.ts` | should not fire for a caught fetch rejected by a pending navigation |
 
 ### ES — Eval on selector