1550813 - Revise viewport-resize-event-on-load-overflowing-page.html to avoid bug 1547827

Status: RESOLVED FIXED

(Core :: Layout, task, P3)

Description

[Botond Ballo [:botond]]

The viewport-resize-event-on-load-overflowing-page.html web platform test currently fails due to bug 1547827.

James suggested that the test could be revised to avoid this issue by using window.open and running the test in the new window instead.

Comment 1

[Botond Ballo [:botond]]

Is there an example of another test that delegates its work to a helper file that I can follow?

Comment 2

[James Graham [:jgraham]]

There's [1] which just does everthing in terms of opener, but there's an alternate approach (which I can't see an example of right now) where you define the tests in the original window, define an event listener for messages, and basically just postMessage the results from the opened window.

[1] https://searchfox.org/mozilla-central/source/testing/web-platform/tests/html/browsers/browsing-the-web/history-traversal/browsing_context_name.html

Comment 3

[Hiroyuki Ikezoe (:hiro)]

I suppose Botond might want to fix this along with shipping the visual viewport API ship (it's 68), but for now I am going to set this P3 since this fix is only for the test case we can easily uplift to the 68 branch.

Comment 4

[Botond Ballo [:botond]]

Attachment: Bug 1550813 - Revise viewport-resize-event-on-load-overflowing-page.html to avoid bug 1547827. r=jgraham

The test still fails, but now the failure is an issue with the feature rather
than the harness.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/37ca86f0b9ca
Revise viewport-resize-event-on-load-overflowing-page.html to avoid bug 1547827. r=jgraham

Comment 6

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

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/16964 for changes under testing/web-platform/tests

Comment 7

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

Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.

Comment 8

[Daniel Varga [:dvarga]]

https://hg.mozilla.org/mozilla-central/rev/37ca86f0b9ca

Comment 9

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

Upstream PR merged