Closed Bug 1302128 Opened 4 years ago Closed 4 years ago
_bug1277814 .html appears to be defective
Bug 1302128 - Fix a test so that it actually exercises the APZ layerization codepath it intends to exericse.
58 bytes, text/x-review-board-request
I have some local patches for bug 1289432 which caused test_bug1277814 to start failing pretty consistently. Looking at the test I see a couple of issues with it: 1) what looks like a typo/oversight: moveMouseAndScrollWheelOver is async in that it queues up the mouse events, but the test continues without waiting for them to actually happen. i.e. it doesn't "yield" the return from that function, it just continues right through. If I change that to yield, then: 2) test hangs because the moveMouseAndScrollWheelOver waits for a scroll event to fire, but no such scroll event ever fires. It looks like this happens because the CSS transform change is animated, and the wheel event happens before the animation is done, so before either the APZ or the main thread knows that there is a scrollable subframe there.
Priority: -- → P3
Comment on attachment 8790303 [details] Bug 1302128 - Fix a test so that it actually exercises the APZ layerization codepath it intends to exericse. https://reviewboard.mozilla.org/r/78192/#review76710 Thanks for catching and fixing these issues!
Attachment #8790303 - Flags: review?(botond) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/9679b6eb4bee Fix a test so that it actually exercises the APZ layerization codepath it intends to exericse. r=botond
You need to log in before you can comment on or make changes to this bug.