1720654 - Make full screen tests wait for the right content page to load in newly-opened windows

Status: RESOLVED FIXED

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

Description

[Henri Sivonen (:hsivonen) (on vacation)]

Address the problem for which bug 1719659 was the wrong suggestion.

Comment 1

[Henri Sivonen (:hsivonen) (on vacation)]

Attachment: Bug 1720654 - Make fullscreen tests wait for the right page to load in newly-opened windows.

Comment 2

[Henri Sivonen (:hsivonen) (on vacation)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=9f12738260899fe6eb3f9eaf73997d8ad100a4fd

Comment 3

[Henri Sivonen (:hsivonen) (on vacation)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=fe3d774c8a7967d1235fec0bb92bda2944a511cc

Comment 4

[Henri Sivonen (:hsivonen) (on vacation)]

(In reply to Henri Sivonen (:hsivonen) from comment #3)

https://treeherder.mozilla.org/#/jobs?repo=try&revision=fe3d774c8a7967d1235fec0bb92bda2944a511cc

https://treeherder.mozilla.org/jobs?repo=try&revision=fe3d774c8a7967d1235fec0bb92bda2944a511cc&selectedTaskRun=OUB-3sQ7QNmThrEIe_SOBQ.0 shows the failure mode that the pause introduced in bug 1698036 is supposed to avoid.

I'm shy to remove the pause. :-(

Sadly, this doesn't make the other failure mode go away, either. This still seems like a correct step to take.

Comment 5

[Henri Sivonen (:hsivonen) (on vacation)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=6f7068ec6d1df9ac11a48761084217cdd9afe2e0

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/81d8b13e2bbe
Make fullscreen tests wait for the right page to load in newly-opened windows. r=nika

Comment 7

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/81d8b13e2bbe