Closed Bug 1602626 Opened 4 years ago Closed 4 years ago

ubuntu1804 - widget/tests/test_wheeltransaction.xhtml | wrong view was scrolled: Continuous scrolling test for sub view 3 (horizontal/forward) - got "rootview", expected "subview3"

Categories

(Core :: DOM: UI Events & Focus Handling, defect, P5)

Product:
Core
Component:
DOM: UI Events & Focus Handling
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla74
Tracking Flags:
Tracking Status
firefox74 --- fixed

People

(Reporter: intermittent-bug-filer, Assigned: masayuki)

References

Details

Attachments

(1 file)

Bug 1602626 - Make window_wheeltransaction.xhtml compute point to fire wheel events dynamically r=smaug
47 bytes, text/x-phabricator-request
Details | Review

Filed by: egao [at] mozilla.com
Parsed log: https://treeherder.mozilla.org/logviewer.html#?job_id=280332609&repo=try
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/I-LQh1gERma0_Xnsi0-Kzw/runs/0/artifacts/public/logs/live_backing.log

suite: mochitest-chrome-1proc
chunk: 2

context:
[task 2019-12-09T18:47:35.034Z] 18:47:35 INFO - TEST-PASS | widget/tests/test_wheeltransaction.xhtml | failed to scroll on current target: Continuous scrolling test for sub view 3 (vertical/forward)
[task 2019-12-09T18:47:35.034Z] 18:47:35 INFO - TEST-PASS | widget/tests/test_wheeltransaction.xhtml | passed: Continuous scrolling test for sub view 3 (vertical/backward)
[task 2019-12-09T18:47:35.035Z] 18:47:35 INFO - TEST-PASS | widget/tests/test_wheeltransaction.xhtml | passed: Continuous scrolling test for sub view 3 (vertical/backward)
[task 2019-12-09T18:47:35.036Z] 18:47:35 INFO - TEST-PASS | widget/tests/test_wheeltransaction.xhtml | passed: Continuous scrolling test for sub view 3 (vertical/backward)
[task 2019-12-09T18:47:35.036Z] 18:47:35 INFO - TEST-PASS | widget/tests/test_wheeltransaction.xhtml | failed to scroll on current target: Continuous scrolling test for sub view 3 (vertical/backward)
[task 2019-12-09T18:47:35.036Z] 18:47:35 INFO - Buffered messages finished
[task 2019-12-09T18:47:35.036Z] 18:47:35 INFO - TEST-UNEXPECTED-FAIL | widget/tests/test_wheeltransaction.xhtml | wrong view was scrolled: Continuous scrolling test for sub view 3 (horizontal/forward) - got "rootview", expected "subview3"
[task 2019-12-09T18:47:35.036Z] 18:47:35 INFO - SimpleTest.is@chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:325:16
[task 2019-12-09T18:47:35.036Z] 18:47:35 INFO - is@chrome://mochitests/content/chrome/widget/tests/window_wheeltransaction.xhtml:82:34
[task 2019-12-09T18:47:35.036Z] 18:47:35 INFO - onScrollView@chrome://mochitests/content/chrome/widget/tests/window_wheeltransaction.xhtml:1376:7
[task 2019-12-09T18:47:35.037Z] 18:47:35 INFO - onscroll@chrome://mochitests/content/chrome/widget/tests/window_wheeltransaction.xhtml:1:13
[task 2019-12-09T18:47:35.038Z] 18:47:35 INFO - _sendWheelAndPaint/onwheel/<@chrome://mochikit/content/tests/SimpleTest/EventUtils.js:657:13
[task 2019-12-09T18:47:35.038Z] 18:47:35 INFO - setTimeout handlerSimpleTest_setTimeoutShim@chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:689:43
[task 2019-12-09T18:47:35.038Z] 18:47:35 INFO - onwheel@chrome://mochikit/content/tests/SimpleTest/EventUtils.js:656:15
[task 2019-12-09T18:47:35.038Z] 18:47:35 INFO - synthesizeWheelAtPoint@chrome://mochikit/content/tests/SimpleTest/EventUtils.js:593:9
[task 2019-12-09T18:47:35.038Z] 18:47:35 INFO - synthesizeWheel@chrome://mochikit/content/tests/SimpleTest/EventUtils.js:620:25
[task 2019-12-09T18:47:35.038Z] 18:47:35 INFO - _sendWheelAndPaint@chrome://mochikit/content/tests/SimpleTest/EventUtils.js:683:20
[task 2019-12-09T18:47:35.038Z] 18:47:35 INFO - sendWheelAndPaint@chrome://mochikit/content/tests/SimpleTest/EventUtils.js:704:21
[task 2019-12-09T18:47:35.039Z] 18:47:35 INFO - _fireWheelScrollEvent@chrome://mochitests/content/chrome/widget/tests/window_wheeltransaction.xhtml:1111:20
[task 2019-12-09T18:47:35.039Z] 18:47:35 INFO - testContinuousScroll@chrome://mochitests/content/chrome/widget/tests/window_wheeltransaction.xhtml:1235:24
[task 2019-12-09T18:47:35.039Z] 18:47:35 INFO - setTimeout handlerSimpleTest_setTimeoutShim@chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:689:43
[task 2019-12-09T18:47:35.039Z] 18:47:35 INFO - runTestStepAt@chrome://mochitests/content/chrome/widget/tests/window_wheeltransaction.xhtml:1516:14
[task 2019-12-09T18:47:35.040Z] 18:47:35 INFO - runNextTestStep@chrome://mochitests/content/chrome/widget/tests/window_wheeltransaction.xhtml:1483:18
[task 2019-12-09T18:47:35.040Z] 18:47:35 INFO - clearWheelTransaction@chrome://mochitests/content/chrome/widget/tests/window_wheeltransaction.xhtml:1050:3
[task 2019-12-09T18:47:35.041Z] 18:47:35 INFO - setTimeout handlerSimpleTest_setTimeoutShim@chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:689:43
[task 2019-12-09T18:47:35.041Z] 18:47:35 INFO - runTestStepAt@chrome://mochitests/content/chrome/widget/tests/window_wheeltransaction.xhtml:1516:14
[task 2019-12-09T18:47:35.042Z] 18:47:35 INFO - runNextTestStep@chrome://mochitests/content/chrome/widget/tests/window_wheeltransaction.xhtml:1483:18
[task 2019-12-09T18:47:35.042Z] 18:47:35 INFO - initElements@chrome://mochitests/content/chrome/widget/tests/window_wheeltransaction.xhtml:1043:3
[task 2019-12-09T18:47:35.043Z] 18:47:35 INFO - setTimeout handlerSimpleTest_setTimeoutShim@chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:689:43
[task 2019-12-09T18:47:35.043Z] 18:47:35 INFO - runTestStepAt@chrome://mochitests/content/chrome/widget/tests/window_wheeltransaction.xhtml:1516:14
[task 2019-12-09T18:47:35.043Z] 18:47:35 INFO - runNextTestList@chrome://mochitests/content/chrome/widget/tests/window_wheeltransaction.xhtml:1501:16
[task 2019-12-09T18:47:35.043Z] 18:47:35 INFO - runTestStepAt@chrome://mochitests/content/chrome/widget/tests/window_wheeltransaction.xhtml:1521:5
[task 2019-12-09T18:47:35.043Z] 18:47:35 INFO - runNextTestStep@chrome://mochitests/content/chrome/widget/tests/window_wheeltransaction.xhtml:1483:18
[task 2019-12-09T18:47:35.043Z] 18:47:35 INFO - testContinuousScroll@chrome://mochitests/content/chrome/widget/tests/window_wheeltransaction.xhtml:1222:5
[task 2019-12-09T18:47:35.044Z] 18:47:35 INFO - setTimeout handlerSimpleTest_setTimeoutShim@chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:689:43
[task 2019-12-09T18:47:35.044Z] 18:47:35 INFO - continueTest@chrome://mochitests/content/chrome/widget/tests/window_wheeltransaction.xhtml:1555:14
[task 2019-12-09T18:47:35.044Z] 18:47:35 INFO - onMouseScrollFailed@chrome://mochitests/content/chrome/widget/tests/window_wheeltransaction.xhtml:1422:3
[task 2019-12-09T18:47:35.044Z] 18:47:35 INFO - synthesizeWheelAtPoint@chrome://mochikit/content/tests/SimpleTest/EventUtils.js:593:9
[task 2019-12-09T18:47:35.044Z] 18:47:35 INFO - synthesizeWheel@chrome://mochikit/content/tests/SimpleTest/EventUtils.js:620:25
[task 2019-12-09T18:47:35.044Z] 18:47:35 INFO - _sendWheelAndPaint@chrome://mochikit/content/tests/SimpleTest/EventUtils.js:683:20
[task 2019-12-09T18:47:35.044Z] 18:47:35 INFO - sendWheelAndPaint@chrome://mochikit/content/tests/SimpleTest/EventUtils.js:704:21
[task 2019-12-09T18:47:35.044Z] 18:47:35 INFO - _fireWheelScrollEvent@chrome://mochitests/content/chrome/widget/tests/window_wheeltransaction.xhtml:1111:20
[task 2019-12-09T18:47:35.044Z] 18:47:35 INFO - testContinuousScroll@chrome://mochitests/content/chrome/widget/tests/window_wheeltransaction.xhtml:1235:24
[task 2019-12-09T18:47:35.044Z] 18:47:35 INFO - setTimeout handlerSimpleTest_setTimeoutShim@chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:689:43
[task 2019-12-09T18:47:35.044Z] 18:47:35 INFO - continueTest@chrome://mochitests/content/chrome/widget/tests/window_wheeltransaction.xhtml:1555:14
[task 2019-12-09T18:47:35.044Z] 18:47:35 INFO - onScrollView@chrome://mochitests/content/chrome/widget/tests/window_wheeltransaction.xhtml:1394:3
[task 2019-12-09T18:47:35.044Z] 18:47:35 INFO - onscroll@chrome://mochitests/content/chrome/widget/tests/window_wheeltransaction.xhtml:1:13
[task 2019-12-09T18:47:35.044Z] 18:47:35 INFO - _sendWheelAndPaint/onwheel/<@chrome://mochikit/content/tests/SimpleTest/EventUtils.js:657:13
[task 2019-12-09T18:47:35.045Z] 18:47:35 INFO - setTimeout handlerSimpleTest_setTimeoutShim@chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:689:43
[task 2019-12-09T18:47:35.045Z] 18:47:35 INFO - onwheel@chrome://mochikit/content/tests/SimpleTest/EventUtils.js:656:15
[task 2019-12-09T18:47:35.045Z] 18:47:35 INFO - synthesizeWheelAtPoint@chrome://mochikit/content/tests/SimpleTest/EventUtils.js:593:9
[task 2019-12-09T18:47:35.045Z] 18:47:35 INFO - synthesizeWheel@chrome://mochikit/content/tests/SimpleTest/EventUtils.js:620:25
[task 2019-12-09T18:47:35.045Z] 18:47:35 INFO - _sendWheelAndPaint@chrome://mochikit/content/tests/SimpleTest/EventUtils.js:683:20
[task 2019-12-09T18:47:35.045Z] 18:47:35 INFO - sendWheelAndPaint@chrome://mochikit/content/tests/SimpleTest/EventUtils.js:704:21
[task 2019-12-09T18:47:35.045Z] 18:47:35 INFO - _fireWheelScrollEvent@chrome://mochitests/content/chrome/widget/tests/window_wheeltransaction.xhtml:1111:20
[task 2019-12-09T18:47:35.045Z] 18:47:35 INFO - testContinuousScroll@chrome://mochitests/content/chrome/widget/tests/window_wheeltransaction.xhtml:1235:24
[task 2019-12-09T18:47:35.045Z] 18:47:35 INFO - setTimeout handlerSimpleTest_setTimeoutShim@chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:689:43
[task 2019-12-09T18:47:35.045Z] 18:47:35 INFO - continueTest@chrome://mochitests/content/chrome/widget/tests/window_wheeltransaction.xhtml:1555:14
[task 2019-12-09T18:47:35.045Z] 18:47:35 INFO - TEST-PASS | widget/tests/test_wheeltransaction.xhtml | passed: Don't reset transaction by a different direction wheel event (1-1)
[task 2019-12-09T18:47:35.045Z] 18:47:35 INFO - TEST-PASS | widget/tests/test_wheeltransaction.xhtml | passed: Don't reset transaction by a different direction wheel event (1-2)
[task 2019-12-09T18:47:35.051Z] 18:47:35 INFO - TEST-PASS | widget/tests/test_wheeltransaction.xhtml | passed: Don't reset transaction by a different direction wheel event (1-3)
[task 2019-12-09T18:47:35.079Z] 18:47:35 INFO - TEST-PASS | widget/tests/test_wheeltransaction.xhtml | passed: Don't reset transaction by a different direction wheel event (2-1)
[task 2019-12-09T18:47:35.095Z] 18:47:35 INFO - TEST-PASS | widget/tests/test_wheeltransaction.xhtml | passed: Don't reset transaction by a different direction wheel event (2-2)
[task 2019-12-09T18:47:35.110Z] 18:47:35 INFO - TEST-PASS | widget/tests/test_wheeltransaction.xhtml | passed: Don't reset transaction by a different direction wheel event (2-3)
[task 2019-12-09T18:47:35.131Z] 18:47:35 INFO - TEST-PASS | widget/tests/test_wheeltransaction.xhtml | passed: Don't reset transaction even if a wheel event cannot scroll (1-1)

:hsinyi - this is a new permafail that is observed when test is run inside ubuntu1804.

Currently, the expected timeline for migrating to ubuntu1804 is end of December 2019. Could you please pass the ni to someone that could take a look at this failure and determine if the test should be fixed or marked with a skip-if/fail-if on ubuntu1804?

Flags: needinfo?(htsai)

Would you please help us here, Masayuki?

Flags: needinfo?(htsai) → needinfo?(masayuki)

I am proposing that we mark this test with a fail-if = os == 'linux' && os_version == '18.04' notation, so we continue to have some coverage and will know if the results change to a pass.

This is really important test because the feature can be broken easily. So, this shouldn't be disabled.

I think that kPtInSubView3ForH is invalid because this is referred only by this test.

Flags: needinfo?(masayuki)

Hmm... I cannot reproduce this on my environment (on VMware).

I wonder if it's something to do with the environment difference (VM vs docker), or difference in the desktop environment.

Are there any pieces of information I can help you get?

You can push to try and run on ubuntu1804 image using ./mach try fuzzy --ubuntu-bionic and selecting the test-linux64 jobs as normal.

Flags: needinfo?(masayuki)

(In reply to Edwin Takahashi (:egao, :etakahashi) from comment #8)

I wonder if it's something to do with the environment difference (VM vs docker), or difference in the desktop environment.

Are there any pieces of information I can help you get?

Honestly, I have no idea. According to the symptom, it fails to consider whether gSubView3 is still scrollable or not. Could be rounding error, but I'm not sure.

You can push to try and run on ubuntu1804 image using ./mach try fuzzy --ubuntu-bionic and selecting the test-linux64 jobs as normal.

Thanks for the information!!

Flags: needinfo?(masayuki)

ni? myself for not forgetting to handle this when I'm back from vacation.

Flags: needinfo?(masayuki)

Okay, looks like that, we meet a round error.

[task 2020-01-08T10:12:41.078Z] 10:12:41 INFO - offset: 270, 30, expectedView: 279, 29, 379, 129.5

Assignee: nobody → masayuki
Status: NEW → ASSIGNED
Flags: needinfo?(masayuki)

Hi :masayuki - checking in on the status of this bug. It is still consistently reproducible on ubuntu1804 image.

Recent try push: https://treeherder.mozilla.org/#/jobs?repo=try&group_state=expanded&tier=1%2C2%2C3&revision=d1ec226d6b6f49aecbbb948e99a2f0bfc97eb6a7&selectedJob=284958383

Yep, sorry for the delay. Currently this is in my queue, but the queue is long due to new year holidays and business trip. I'll post a patch soon.

Thanks for reply - and no worries. I realized that there are two other blockers in migrating mochitest-chrome to ubuntu1804, so even if you get to this bug next second I'm still blocked.

Please take your time and thanks for looking into it.

Currently, offset in root view in the test is copied from constants which are
computed from CSS rules. However, the point may be edge of each view and
that might be outside if it meets round error of layout. For making this
test more stable in any environments, this patch makes the test compute
point to fire wheel events dynamically.

Pushed by masayuki@d-toybox.com:
https://hg.mozilla.org/integration/autoland/rev/4380016a4178
Make window_wheeltransaction.xhtml compute point to fire wheel events dynamically r=smaug

https://hg.mozilla.org/mozilla-central/rev/4380016a4178

Status: ASSIGNED → RESOLVED
Closed: 4 years ago
status-firefox74: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla74
Attachment #9121493 - Attachment description: Bug 1602626 - Make window_wheeltransaction.xhtml compute point to fire wheel events dynamically r=smaug! → Bug 1602626 - Make window_wheeltransaction.xhtml compute point to fire wheel events dynamically r=smaug
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: