[Docs]: incorrect defaults and broken code samples in several guides

[ashrafiucse]

[Docs]: incorrect defaults and broken code samples in several guides

Page(s)

• https://playwright.dev/docs/api/class-browsertype#browsertype-launch-option-timeout (and every page embedding %%-browser-option-timeout%%)
• https://playwright.dev/docs/dialogs
• https://playwright.dev/docs/handles
• https://playwright.dev/docs/input
• https://playwright.dev/docs/service-workers
• https://playwright.dev/docs/test-configuration
• https://playwright.dev/docs/test-parallel
• https://playwright.dev/docs/test-reporters

Description

Several guides show incorrect defaults or broken samples:

1. Browser launch timeout default is wrong. Docs say timeout "Defaults to 30000 (30 seconds)". The actual default is 180000 (3 minutes) — see DEFAULT_PLAYWRIGHT_LAUNCH_TIMEOUT in packages/isomorphic/time.ts.
2. dialogs.md, Python async sample: misplaced parenthesis — page.get_by_role("button".click()) should be page.get_by_role("button").click().
3. dialogs.md, beforeunload Python samples: the handler is passed as a zero-argument lambda — page.on('dialog', lambda: handle_dialog) registers a lambda that returns the handler instead of calling it; should be page.on('dialog', handle_dialog).
4. handles.md, Java/C# samples: the evaluate callback runs in the page, but uses List.add (arg.myArray.add(arg.newElement)); should be Array.push (arg.myArray.push(arg.newElement)).
5. input.md, C# time input sample: invalid value "13-15"; the correct time format is "13:15".
6. service-workers (JS/Python): the text says "set serviceWorkers to 'block'" but the sample config uses 'allow'.
7. test-parallel: says worker index "starts with 1", but workerHost.ts initializes lastWorkerIndex = 0 and assigns workerIndex = lastWorkerIndex++ — worker index starts with 0.
8. test-reporters, Blob reporter: documented default file name is report-<project>-<hash>-<shard_number>.zip; the actual default is report-<hash>-<shard_number>.zip with no project segment (see _defaultReportName in packages/playwright/src/reporters/blob.ts).
9. test-configuration, testMatch: default shown as regex .*(test|spec).(js|ts|mjs); the actual default glob is **/*.@(spec|test).?(c|m)[jt]s?(x) (see packages/playwright/src/common/config.ts).

I'd like to work on this myself.

[dgozman]

Ashraf Ali (@ashrafiucse) Thank you for the issue, please send a PR now.

[ashrafiucse]

I actually have one already — #42519, closed before the issue existed. Fine to reopen that if it's easier than a fresh PR.