1499239 - Intermittent mobile/android/tests/browser/chrome/test_session_scroll_position.html | scrollY set correctly - got 292, expected 343

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: Testing, defect)

Description

[Treeherder Bug Filer]

Filed by: aciure [at] mozilla.com

https://treeherder.mozilla.org/logviewer.html#?job_id=205631970&repo=autoland

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

[task 2018-10-15T22:20:33.642Z] 22:20:33     INFO -  158 INFO TEST-PASS | mobile/android/tests/browser/chrome/test_session_scroll_position.html | scrollX set correctly
[task 2018-10-15T22:20:33.642Z] 22:20:33     INFO -  Buffered messages finished
[task 2018-10-15T22:20:33.643Z] 22:20:33     INFO -  159 INFO TEST-UNEXPECTED-FAIL | mobile/android/tests/browser/chrome/test_session_scroll_position.html | scrollY set correctly - got 292, expected 343
[task 2018-10-15T22:20:33.644Z] 22:20:33     INFO -  SimpleTest.is@chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:312:5
[task 2018-10-15T22:20:33.644Z] 22:20:33     INFO -  checkScroll@chrome://mochitests/content/chrome/mobile/android/tests/browser/chrome/test_session_scroll_position.html:86:7
[task 2018-10-15T22:20:33.645Z] 22:20:33     INFO -  createAndRemoveTab/<@chrome://mochitests/content/chrome/mobile/android/tests/browser/chrome/test_session_scroll_position.html:277:9
[task 2018-10-15T22:20:33.646Z] 22:20:33     INFO -  TaskImpl_run@resource://gre/modules/Task.jsm:326:42
[task 2018-10-15T22:20:33.646Z] 22:20:33     INFO -  160 INFO TEST-PASS | mobile/android/tests/browser/chrome/test_session_scroll_position.html | zoom set correctly
[task 2018-10-15T22:20:33.647Z] 22:20:33     INFO -  161 INFO Now waiting for SSTabScrollCaptured event from container
[task 2018-10-15T22:20:33.647Z] 22:20:33     INFO -  162 INFO Received event SSTabScrollCaptured from container
[task 2018-10-15T22:20:33.648Z] 22:20:33     INFO -  163 INFO TEST-PASS | mobile/android/tests/browser/chrome/test_session_scroll_position.html | scrollX set correctly
[task 2018-10-15T22:20:33.648Z] 22:20:33     INFO -  164 INFO TEST-PASS | mobile/android/tests/browser/chrome/test_session_scroll_position.html | scrollY set correctly
[task 2018-10-15T22:20:33.649Z] 22:20:33     INFO -  165 INFO Now waiting for SSTabScrollCaptured event from container
[task 2018-10-15T22:20:33.650Z] 22:20:33     INFO -  166 INFO Received event SSTabScrollCaptured from container
[task 2018-10-15T22:20:33.651Z] 22:20:33     INFO -  167 INFO TEST-PASS | mobile/android/tests/browser/chrome/test_session_scroll_position.html | scrollX set correctly
[task 2018-10-15T22:20:33.651Z] 22:20:33     INFO -  168 INFO TEST-PASS | mobile/android/tests/browser/chrome/test_session_scroll_position.html | scrollY set correctly
[task 2018-10-15T22:20:33.652Z] 22:20:33     INFO -  169 INFO Now waiting for SSTabCloseProcessed event from container
[task 2018-10-15T22:20:33.653Z] 22:20:33     INFO -  170 INFO Received event SSTabCloseProcessed from container
[task 2018-10-15T22:20:33.654Z] 22:20:33     INFO -  171 INFO TEST-UNEXPECTED-FAIL | mobile/android/tests/browser/chrome/test_session_scroll_position.html | stored scroll position for frameset is correct - got "0,292", expected "0,343"
[task 2018-10-15T22:20:33.654Z] 22:20:33     INFO -  SimpleTest.is@chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:312:5
[task 2018-10-15T22:20:33.655Z] 22:20:33     INFO -  test_sessionStoreScrollPositionForFrames@chrome://mochitests/content/chrome/mobile/android/tests/browser/chrome/test_session_scroll_position.html:298:5
[task 2018-10-15T22:20:33.656Z] 22:20:33     INFO -  _handleTabClosed@chrome://browser/content/browser.js:1272:5
[task 2018-10-15T22:20:33.657Z] 22:20:33     INFO -  onEvent@chrome://browser/content/browser.js:1997:9

Comment 1

[Jan Henning [:JanH]]

Oops, seems like the frameset scroll that much. Fix coming this evening.

Comment 2

[Jan Henning [:JanH]]

(In reply to Jan Henning [:JanH] from comment #1)
> Oops, seems like the frameset scroll that much.

... like the frameset *can't* scroll that much.

Comment 3

[Jan Henning [:JanH]]

Attachment: Bug 1499239 - Use a smaller scroll range for the frameset. r?gbrown

The size of the frameset on the emulator is smaller than I expected, so we can't
scroll the full range of SCROLL_Y even when we've zoomed in somewhat.
Because SCROLL_Y has a maximum of 400 and the scroll got truncated to 292, we
divide the number in half.

So we don't have to generate yet another hard-coded scroll string for this case,
we switch to generating them on the fly from the test data for the respective
sub test.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/de6fb9a3d7a5
Use a smaller scroll range for the frameset. r=gbrown

Comment 5

[Stefan Hindli [:stefan_hindli]]

https://hg.mozilla.org/mozilla-central/rev/de6fb9a3d7a5

Comment 6

[Intermittent Failures Robot]

63 failures in 515 pushes (0.122 failures/push) were associated with this bug yesterday.

Repository breakdown:
* mozilla-central: 2
* try: 1
* mozilla-inbound: 23
* autoland: 37

Platform breakdown:
* android-em-4-2-x86: 63

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=1499239&startday=2018-10-16&endday=2018-10-16&tree=all

Comment 7

[Robert AB]

Is this bug fixed?
I have noticed error in comment 6.

Comment 8

[Jan Henning [:JanH]]

Those errors all happened in the period between bug 1498892 landing and this fix.

Comment 9

[Intermittent Failures Robot]

69 failures in 2759 pushes (0.025 failures/push) were associated with this bug in the last 7 days.

This is the #29 most frequent failure this week.

** This failure happened more than 30 times this week! Resolving this bug is a high priority. **

** Try to resolve this bug as soon as possible. If unresolved for 2 weeks, the affected test(s) may be disabled. ** 

Repository breakdown:
* mozilla-inbound: 23
* autoland: 43
* mozilla-central: 2
* try: 1

Platform breakdown:
* android-em-4-2-x86: 69

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=1499239&startday=2018-10-15&endday=2018-10-21&tree=all

Comment 10

[Intermittent Failures Robot]

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

Repository breakdown:

• try: 1

Platform breakdown:

• android-em-4-3-armv7-api16: 1

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