382283 - Implement "accelerated" scrolling for Gtk2 for page with position:fixed elements.

Status: RESOLVED INVALID

(Core :: Web Painting, defect)

Description

[Stephen Moehle]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9a5pre) Gecko/20070528 Firefox/3.0a5pre
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9a5pre) Gecko/20070528 Firefox/3.0a5pre

Implement the same type of scrolling under Gtk2 (Linux, Sun, BSD, etc.) that was done for Windows in bug 343430 for pages with postion:fixed elements. Basically, implement the clipping rectangle in nsWindow::Scroll() for Gtk2.

Unfortunately, the patch I have uses gdk_windows_move_region() which is only available in Gtk2 2.8, which is higher than the current minimum requirement. So this fix cannot be used until http://wiki.mozilla.org/Linux/Runtime_Requirements, or something close, is adopted.

Reproducible: Always

Comment 1

[Stephen Moehle]

Attachment: Optimize scrolling for Gtk2 for pages with postion:fixed elements

Implement the clipping rect for nsWindow::Scroll() for Gtk2. Make view changes for bug 343430 apply to Gtk2 as well as Windows. Implement nsWindow::ScrollRect() for Gtk2 because it was there.

Comment 2

[Stephen Moehle]

All of the test cases in bug 343430, bug 324819, and their various duplicates all  work fine as best as I can tell with this patch.

Comment 3

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Reassigning to you ... don't forget to request review when those runtime requirements are adopted (I think they will become official sometime soon)

Comment 4

[Stephen Moehle]

Attachment: better fix

Better fix. Make dropdown lists in the Firefox UI, such as fonts on the Preferences dialog, scroll properly. See the changes to nsViewManager::UpdateViewAfterScroll().

Comment 5

[Jonathan Watt [:jwatt]]

Closing GTK2 related bugs since we removed GTK2 support at the beginning of 2018 in bug 1278282. Probably best to open a new bug in the unlikely event that any of these are still relevant.