1524220 - Perma /intersection-observer/root-margin.html | document.scrollingElement.scrollTop = document.documentElement.clientHeight + 200 - assert_equals: entries.length expected 2 but got 1

Status: RESOLVED FIXED

(Core :: Layout, defect, P5)

Description

[Treeherder Bug Filer]

#[markdown(off)]
Filed by: rmaries [at] mozilla.com

https://treeherder.mozilla.org/logviewer.html#?job_id=225182702&repo=mozilla-central

https://queue.taskcluster.net/v1/task/bny27pMpQ5SReL8N6hKXJQ/runs/0/artifacts/public/logs/live_backing.log

[task 2019-01-31T10:30:19.036Z] 10:30:19 INFO - TEST-START | /intersection-observer/root-margin.html
[task 2019-01-31T10:30:19.547Z] 10:30:19 INFO - Closing window 103
[task 2019-01-31T10:30:19.584Z] 10:30:19 INFO -

[task 2019-01-31T10:30:19.590Z] 10:30:19 INFO - Test.prototype.step_timeout/<@http://web-platform.test:8000/resources/testharness.js:1624:20
[task 2019-01-31T10:30:19.590Z] 10:30:19 INFO - Test.prototype.step@http://web-platform.test:8000/resources/testharness.js:1566:20
[task 2019-01-31T10:30:19.590Z] 10:30:19 INFO - Test.prototype.step_func/<@http://web-platform.test:8000/resources/testharness.js:1590:20
[task 2019-01-31T10:30:19.590Z] 10:30:19 INFO - TEST-UNEXPECTED-FAIL | /intersection-observer/root-margin.html | document.scrollingElement.scrollTop = document.documentElement.clientHeight + 200 - assert_equals: entries.length expected 2 but got 1
[task 2019-01-31T10:30:19.590Z] 10:30:19 INFO - checkLastEntry@http://web-platform.test:8000/intersection-observer/resources/intersection-observer-test-utils.js:133:3
[task 2019-01-31T10:30:19.590Z] 10:30:19 INFO - step2@http://web-platform.test:8000/intersection-observer/root-margin.html:77:3
[task 2019-01-31T10:30:19.590Z] 10:30:19 INFO - Test.prototype.step@http://web-platform.test:8000/resources/testharness.js:1566:20
[task 2019-01-31T10:30:19.590Z] 10:30:19 INFO - Test.prototype.step_func_done/<@http://web-platform.test:8000/resources/testharness.js:1606:17
[task 2019-01-31T10:30:19.590Z] 10:30:19 INFO - Test.prototype.step_timeout/<@http://web-platform.test:8000/resources/testharness.js:1624:20
[task 2019-01-31T10:30:19.590Z] 10:30:19 INFO - Test.prototype.step@http://web-platform.test:8000/resources/testharness.js:1566:20
[task 2019-01-31T10:30:19.590Z] 10:30:19 INFO - Test.prototype.step_func/<@http://web-platform.test:8000/resources/testharness.js:1590:20
[task 2019-01-31T10:30:19.590Z] 10:30:19 INFO - .

Comment 1

[Hiroyuki Ikezoe (:hiro)]

There is a suspicious commit which is mine. :/

Comment 2

[Cristina Coroiu [:ccoroiu]]

Started to fail with this merge:
https://treeherder.mozilla.org/#/jobs?repo=mozilla-central&selectedJob=225189088&revision=efda4de34c171f6acbb19ea1fd07872745df2760

:hiro, please take a look. Thank you

Comment 3

[Andreea Pavel [:apavel]]

There are 20 total failures in the last 7 days, all on android-em-7-0-x86 opt.

Recent failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=225683903&repo=mozilla-central&lineNumber=2495

[task 2019-02-02T22:58:25.412Z] 22:58:25 INFO - TEST-START | /intersection-observer/remove-element.html
[task 2019-02-02T22:58:25.759Z] 22:58:25 INFO - Closing window 99
[task 2019-02-02T22:58:25.775Z] 22:58:25 INFO - ......
[task 2019-02-02T22:58:25.775Z] 22:58:25 INFO - TEST-OK | /intersection-observer/remove-element.html | took 366ms
[task 2019-02-02T22:58:25.775Z] 22:58:25 INFO - TEST-START | /intersection-observer/root-margin.html
[task 2019-02-02T22:58:26.074Z] 22:58:26 INFO - Closing window 103
[task 2019-02-02T22:58:26.097Z] 22:58:26 INFO -
[task 2019-02-02T22:58:26.097Z] 22:58:26 INFO - TEST-PASS | /intersection-observer/root-margin.html | Root margin tests
[task 2019-02-02T22:58:26.097Z] 22:58:26 INFO - TEST-FAIL | /intersection-observer/root-margin.html | First rAF. - assert_equals: entries[0].intersectionRect.left expected 0 but got 1187
[task 2019-02-02T22:58:26.098Z] 22:58:26 INFO - checkRect@http://web-platform.test:8000/intersection-observer/resources/intersection-observer-test-utils.js:126:3
[task 2019-02-02T22:58:26.098Z] 22:58:26 INFO - checkLastEntry@http://web-platform.test:8000/intersection-observer/resources/intersection-observer-test-utils.js:138:5
[task 2019-02-02T22:58:26.098Z] 22:58:26 INFO - step0@http://web-platform.test:8000/intersection-observer/root-margin.html:58:3
[task 2019-02-02T22:58:26.098Z] 22:58:26 INFO - Test.prototype.step@http://web-platform.test:8000/resources/testharness.js:1566:20
[task 2019-02-02T22:58:26.098Z] 22:58:26 INFO - Test.prototype.step_func_done/<@http://web-platform.test:8000/resources/testharness.js:1606:17
[task 2019-02-02T22:58:26.098Z] 22:58:26 INFO - Test.prototype.step_timeout/<@http://web-platform.test:8000/resources/testharness.js:1624:20
[task 2019-02-02T22:58:26.098Z] 22:58:26 INFO - Test.prototype.step@http://web-platform.test:8000/resources/testharness.js:1566:20
[task 2019-02-02T22:58:26.098Z] 22:58:26 INFO - Test.prototype.step_func/<@http://web-platform.test:8000/resources/testharness.js:1590:20
[task 2019-02-02T22:58:26.098Z] 22:58:26 INFO - TEST-FAIL | /intersection-observer/root-margin.html | document.scrollingElement.scrollLeft = 100 - assert_equals: entries.length expected 2 but got 1
[task 2019-02-02T22:58:26.098Z] 22:58:26 INFO - checkLastEntry@http://web-platform.test:8000/intersection-observer/resources/intersection-observer-test-utils.js:133:3
[task 2019-02-02T22:58:26.098Z] 22:58:26 INFO - step1@http://web-platform.test:8000/intersection-observer/root-margin.html:67:3
[task 2019-02-02T22:58:26.098Z] 22:58:26 INFO - Test.prototype.step@http://web-platform.test:8000/resources/testharness.js:1566:20
[task 2019-02-02T22:58:26.099Z] 22:58:26 INFO - Test.prototype.step_func_done/<@http://web-platform.test:8000/resources/testharness.js:1606:17
[task 2019-02-02T22:58:26.099Z] 22:58:26 INFO - Test.prototype.step_timeout/<@http://web-platform.test:8000/resources/testharness.js:1624:20
[task 2019-02-02T22:58:26.099Z] 22:58:26 INFO - Test.prototype.step@http://web-platform.test:8000/resources/testharness.js:1566:20
[task 2019-02-02T22:58:26.099Z] 22:58:26 INFO - Test.prototype.step_func/<@http://web-platform.test:8000/resources/testharness.js:1590:20
[task 2019-02-02T22:58:26.099Z] 22:58:26 INFO - TEST-UNEXPECTED-FAIL | /intersection-observer/root-margin.html | document.scrollingElement.scrollTop = document.documentElement.clientHeight + 200 - assert_equals: entries.length expected 2 but got 1
[task 2019-02-02T22:58:26.099Z] 22:58:26 INFO - checkLastEntry@http://web-platform.test:8000/intersection-observer/resources/intersection-observer-test-utils.js:133:3
[task 2019-02-02T22:58:26.099Z] 22:58:26 INFO - step2@http://web-platform.test:8000/intersection-observer/root-margin.html:77:3
[task 2019-02-02T22:58:26.099Z] 22:58:26 INFO - Test.prototype.step@http://web-platform.test:8000/resources/testharness.js:1566:20
[task 2019-02-02T22:58:26.099Z] 22:58:26 INFO - Test.prototype.step_func_done/<@http://web-platform.test:8000/resources/testharness.js:1606:17
[task 2019-02-02T22:58:26.099Z] 22:58:26 INFO - Test.prototype.step_timeout/<@http://web-platform.test:8000/resources/testharness.js:1624:20
[task 2019-02-02T22:58:26.099Z] 22:58:26 INFO - Test.prototype.step@http://web-platform.test:8000/resources/testharness.js:1566:20
[task 2019-02-02T22:58:26.100Z] 22:58:26 INFO - Test.prototype.step_func/<@http://web-platform.test:8000/resources/testharness.js:1590:20
[task 2019-02-02T22:58:26.100Z] 22:58:26 INFO - .
[task 2019-02-02T22:58:26.100Z] 22:58:26 INFO - TEST-OK | /intersection-observer/root-margin.html | took 321ms

Comment 4

[Hiroyuki Ikezoe (:hiro)]

I am pretty sure that this test also will be fixed by adding minimum-scale=1, before doing it, but I'd like to know how the minimum scale size affects the IntersectionObserver.

Comment 5

[Intermittent Failures Robot]

26 failures in 2628 pushes (0.01 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• try: 2
• mozilla-central: 24

Platform breakdown:

• android-em-7-0-x86: 26

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=1524220&startday=2019-01-28&endday=2019-02-03&tree=all

Comment 6

[Hiroyuki Ikezoe (:hiro)]

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

Comment 7

[Hiroyuki Ikezoe (:hiro)]

As discuss with Cameron and Aryx, I am going to fix the issue on upstream directly, but it would take some time, so I am going to disable this test on Android for now, and once upstream fix gets merged into our repos, I will enable it again.

Comment 8

[Hiroyuki Ikezoe (:hiro)]

I should have added them into CC list.

Comment 9

[Hiroyuki Ikezoe (:hiro)]

Err, I am going to mark this test case fail instead of disabling it so that we don't need to enable it again.

Comment 10

[Hiroyuki Ikezoe (:hiro)]

Sent a PR to the upstream. https://github.com/web-platform-tests/wpt/pull/15254

Comment 11

[Hiroyuki Ikezoe (:hiro)]

Attachment: Bug 1524220 - Mark 'document.documentElement.clientHeight + 200' test case fail. r=heycam

All of failure test cases in root-margin.html will be fixed by this PR
https://github.com/web-platform-tests/wpt/pull/15254 .

Comment 12

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/241a3659869b
Mark 'document.documentElement.clientHeight + 200' test case fail. r=heycam

Comment 13

[Oana Pop-Rus]

https://hg.mozilla.org/mozilla-central/rev/241a3659869b

Comment 14

[Intermittent Failures Robot]

13 failures in 2658 pushes (0.005 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• mozilla-central: 13

Platform breakdown:

• android-em-7-0-x86: 13

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=1524220&startday=2019-02-04&endday=2019-02-10&tree=all