Closed Bug 1561726 Opened 1 year ago Closed 1 year ago

Add a test for dragging the RCD-RSF scrollbar

Categories

(Core :: Panning and Zooming, task)

task
Not set

Tracking

()

RESOLVED FIXED
mozilla69
Tracking Status
firefox69 --- fixed

People

(Reporter: botond, Assigned: botond)

References

(Blocks 1 open bug)

Details

Attachments

(4 files)

Our existing APZ mochitests which exercise scrollbar dragging all use subframe scrollbars. The RCD-RSF scrollbar is unique in some ways (e.g. hit-testing it has interactions with zooming) so we should test dragging it as well.

Dragging the viewport scrollbar is accomplished by passing in a window
rather than an element.

Note that we can't just pass in the window's document.documentElement,
because coordinatesRelativeToScreen() would not give the correct result
for it. This is turn is because for a scrollable <div>, getBoundingClientRect()
returns the scroll frame's outer rect, but for the <html> element,
getBoundingClientRect() returns the root scroll frame's inner rect.

Bugbug thinks this bug is a task, but please change it back in case of error.

Type: defect → task
Pushed by bballo@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/6d20bab59174
Add an nsIContent overload of nsLayoutUtils::FindScrollableFrameFor. r=tnikkel
https://hg.mozilla.org/integration/autoland/rev/13557cb78993
Add an nsIDOMWindowUtils API for querying the size of layout scrollbars. r=tnikkel
https://hg.mozilla.org/integration/autoland/rev/fdaf7d75ccc9
Modify dragVerticalScrollbar() to support dragging the viewport scrollbar as well. r=kats
https://hg.mozilla.org/integration/autoland/rev/76c526cf901b
Add a test to exercise dragging the viewport scrollbar. r=kats
You need to log in before you can comment on or make changes to this bug.