1829248 - [wpt-sync] Sync PR 39636 - beforeunload returnValue is empty string should not prompt confirmation

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, task, P4)

Description

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Sync web-platform-tests PR 39636 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/39636
Details from upstream follow.

Di Zhang <[e-mail]> wrote:

beforeunload returnValue is empty string should not prompt confirmation

By HTML spec, setting the returnValue to be the empty string should not
prompt the confirmation dialog for beforeunload event.

[1] https://html.spec.whatwg.org/multipage/browsing-the-web.html#unloading-documents:event-beforeunload:~:text=the%20returnValue%20attribute%20of%20event%20is%20not%20the%20empty%20string

Fixed: 1434509
Change-Id: Ib509150b471a137c7af50d3557a8230ecaaf7f7e
Reviewed-on: https://chromium-review.googlesource.com/4452046
WPT-Export-Revision: ceeba10f3068e8975049baea4cff95013e734277

Comment 1

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=4939810a9cfd7f525cae1fb1ce218f9000b6bc56

Comment 2

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=17f68075e8ce254d3c68f2392c26d1b4c24044eb

Comment 3

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

CI Results

Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 1 tests and 22 subtests

Status Summary

Firefox

OK : 1
PASS: 22

Chrome

OK : 1
PASS: 22

Safari

OK : 1
PASS: 22

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/39bc72c9b04e
[wpt PR 39636] - beforeunload returnValue is empty string should not prompt confirmation, a=testonly

Comment 5

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/39bc72c9b04e