Changeset 86627 in webkit


Ignore:
Timestamp:
May 16, 2011 4:02:52 PM (13 years ago)
Author:
commit-queue@webkit.org
Message:

2011-05-16 Martin Robinson <mrobinson@igalia.com>

Reviewed by Xan Lopez.

[GTK] Scrolling in Twitter is broken after r86102
https://bugs.webkit.org/show_bug.cgi?id=60922

Fix adjustment handling for pages that trigger the slow scrolling path.
The slow scrolling path is triggered by WebCore when it determines that
doing a simple invalidation is quicker than doing a normal scroll. This
typically happens when there are large elements with fixed positions.

  • WebCoreSupport/ChromeClientGtk.cpp: (WebKit::ChromeClient::invalidateContentsForSlowScroll): Poke the adjustment watcher to update its adjustments when a page triggers the slow scrolling path.
Location:
trunk/Source/WebKit/gtk
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/WebKit/gtk/ChangeLog

    r86584 r86627  
     12011-05-16  Martin Robinson  <mrobinson@igalia.com>
     2
     3        Reviewed by Xan Lopez.
     4
     5        [GTK] Scrolling in Twitter is broken after r86102
     6        https://bugs.webkit.org/show_bug.cgi?id=60922
     7
     8        Fix adjustment handling for pages that trigger the slow scrolling path.
     9        The slow scrolling path is triggered by WebCore when it determines that
     10        doing a simple invalidation is quicker than doing a normal scroll. This
     11        typically happens when there are large elements with fixed positions.
     12
     13        * WebCoreSupport/ChromeClientGtk.cpp:
     14        (WebKit::ChromeClient::invalidateContentsForSlowScroll): Poke the adjustment
     15        watcher to update its adjustments when a page triggers the slow scrolling path.
     16
    1172011-05-13  Jon Lee  <jonlee@apple.com>
    218
  • trunk/Source/WebKit/gtk/WebCoreSupport/ChromeClientGtk.cpp

    r86102 r86627  
    404404{
    405405    invalidateContentsAndWindow(updateRect, immediate);
     406    m_adjustmentWatcher.updateAdjustmentsFromScrollbarsLater();
    406407}
    407408
Note: See TracChangeset for help on using the changeset viewer.