1489671 - [wpt-sync] Sync PR 12901 - Loosen timing expectation in IntersectionObserver WPTs

Status: RESOLVED FIXED

(Core :: Layout, enhancement, P4)

Description

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

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

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

Ali Juma <[e-mail]> wrote:
>  Loosen timing expectation in IntersectionObserver WPTs
>  
>  IntersectionObserver WPTs expect that intersection observations are
>  delivered withing two step_timeouts of requestAnimationFrame. This
>  expectation holds in Blink, but doesn't necessarily hold in other engines.
>  In WebKit, requestAnimationFrame and layout are separate tasks, so it's
>  possible for the first step_timeout to fire in between rAF and layout,
>  leading to test flakiness. This CL makes tests wait for an additional
>  step_timeout to account for this.
>  
>  Change-Id: Iad80943db184bc4519ab323c3bd7cfac1256e175
>  Reviewed-on: https://chromium-review.googlesource.com/1214176
>  WPT-Export-Revision: 5756f6a53bc7f6ad87104c3f3ae855d8647070f3

Comment 1

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

Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=a35dc58342ab74552f462d59883581d9d60091e0

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=31da5dab609e50090286c774657dff2d3c0ea836

Comment 3

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

Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=8980f2162c54e0a91bfea44cbfb5b1bc54a3a20a

Comment 4

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

Ran 25 tests and 103 subtests
OK     : 25
PASS   : 102
FAIL   : 1

Comment 5

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

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

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/075ae9254379
[wpt PR 12901] - Loosen timing expectation in IntersectionObserver WPTs, a=testonly

Comment 7

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/075ae9254379

Comment 8

[Dorel Luca [:dluca]]

Backed out for WPT mass failure at /fullscreen/idlharness.window.html

Backout: https://hg.mozilla.org/mozilla-central/rev/e1a681345945ab8c8b8270afcd2f62bf042226c2

Failure push: 
https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&resultStatus=testfailed,busted,exception&classifiedState=unclassified&revision=be6801cdeb537afa655bbd67d12ead8dc01c6c38

https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=3f3f67d1c0bf5359ac39ddddec5999ceb928e7ca

Failure log: https://treeherder.mozilla.org/logviewer.html#?job_id=199470643&repo=mozilla-central&lineNumber=1487

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/56c514060216
[wpt PR 12901] - Loosen timing expectation in IntersectionObserver WPTs, a=testonly

Comment 10

[Andrei Ciure[:aciure]]

https://hg.mozilla.org/mozilla-central/rev/56c514060216