Closed Bug 1216488 Opened 6 years ago Closed 5 years ago

[APZ] overflow: auto/scroll; would not scroll on scrollber by mouse wheel when apz enabled

Categories

(Core :: Panning and Zooming, defect)

44 Branch
Unspecified
Windows 7
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla44
Tracking Status
firefox44 --- fixed

People

(Reporter: alice0775, Assigned: dvander)

References

Details

(Keywords: regression)

Attachments

(2 files)

Attached file testcase
Build Identifier:
https://hg.mozilla.org/integration/mozilla-inbound/rev/da38bd42a863a6b3228500912a1bb2766ff79166
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:44.0) Gecko/20100101 Firefox/44.0 ID:20151020024007

Steps To Reproduce:
1. Open attached testcase
2. mouse over scrollbar(not thumb) of Box1 and turn mouse wheel one tick
   --- may work as expected if no scrollbar on root element

3. mouse over scrollbar(not thumb) of Box2 and turn mouse wheel one tick
   --- does not work, BUG!!!

4. mouse over scrollbar(not thumb) of Box3 and turn mouse wheel one tick
   --- does not work, BUG!!!
   

Actual Results:
Not scroll the element.
However, It may work 2nd tick

Expected Results:
these elements should scroll properly

Bug 1190112 and Bug 1213324 did not fix the problem.
It works when disable APZ
Attached patch bug1216488.patchSplinter Review
Silly bug in the patch for bug 1213324 prevented this test case from being fixed too. We can't use NULL_SCROLL_ID to check whether we're building a scrollbar frame since that's used when the scrollframe isn't active.
Assignee: nobody → dvander
Status: NEW → ASSIGNED
Attachment #8677099 - Flags: review?(tnikkel)
Attachment #8677099 - Flags: review?(tnikkel) → review+
https://hg.mozilla.org/mozilla-central/rev/24653ad483de
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
You need to log in before you can comment on or make changes to this bug.