1785178 - [wpt-sync] Sync PR 35495 - Don't rely on iframe load event firing after subsequent script executes.

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 35495 into mozilla-central (this bug is closed when the sync is complete).

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

Jeremy Roman <[e-mail]> wrote:

Don't rely on iframe load event firing after subsequent script executes.

It appears that it was possible for PostMessageTest to not yet be
defined when the iframe load events are dispatched. Move them after
the definition of that function so that's impossible, and rewrite the
PostMessageTest function inline in those handlers, since it's quite
short anyhow.

Fixed: 1342518,1343698
Change-Id: Iafe632570b10991c1345fdc26063ebc1a85ed1a7

Reviewed-on: https://chromium-review.googlesource.com/3827819
WPT-Export-Revision: 8fcd519d786575f4ecd09a81a6411ba2434a138b

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=7babfd44b7daf4b83cd8100fe17640b0dc831ee3

Comment 2

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

CI Results

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

Total 2 tests and 1 subtests

Status Summary

Firefox

OK : 2
PASS: 2

Chrome

OK : 2
PASS: 2

Safari

OK : 2
PASS: 2

Links

GitHub PR Head
GitHub PR Base

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/4f4ca04be41a
[wpt PR 35495] - Don't rely on iframe load event firing after subsequent script executes., a=testonly

Comment 4

[Cristina Cozmuta (:CrissCozmuta)]

https://hg.mozilla.org/mozilla-central/rev/4f4ca04be41a