[SCM] WebKit Debian packaging branch, webkit-1.3, updated. upstream/1.3.7-4207-g178b198

tony at chromium.org tony at chromium.org
Sun Feb 20 23:35:24 UTC 2011


The following commit has been merged in the webkit-1.3 branch:
commit c0e73588e6eb569657e643d2d04a776f5592067b
Author: tony at chromium.org <tony at chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date:   Fri Jan 21 22:32:15 2011 +0000

    2011-01-21  Tony Chang  <tony at chromium.org>
    
            Reviewed by Sam Weinig.
    
            reduce number of FrameLoaderClient::didChangeScrollOffset calls
            https://bugs.webkit.org/show_bug.cgi?id=52915
    
            Only notify of changes in scroll offset when there actually is a change.
            This regressed in r76291.
    
            Covered by Chromium browser_tests.
    
            * platform/ScrollAnimator.cpp:
            (WebCore::ScrollAnimator::scrollToOffsetWithoutAnimation):
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@76395 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
index 852dab2..0488da5 100644
--- a/Source/WebCore/ChangeLog
+++ b/Source/WebCore/ChangeLog
@@ -1,3 +1,18 @@
+2011-01-21  Tony Chang  <tony at chromium.org>
+
+        Reviewed by Sam Weinig.
+
+        reduce number of FrameLoaderClient::didChangeScrollOffset calls
+        https://bugs.webkit.org/show_bug.cgi?id=52915
+
+        Only notify of changes in scroll offset when there actually is a change.
+        This regressed in r76291.
+
+        Covered by Chromium browser_tests.
+
+        * platform/ScrollAnimator.cpp:
+        (WebCore::ScrollAnimator::scrollToOffsetWithoutAnimation):
+
 2011-01-21  Pavel Feldman  <pfeldman at chromium.org>
 
         Reviewed by Yury Semikhatsky.
diff --git a/Source/WebCore/platform/ScrollAnimator.cpp b/Source/WebCore/platform/ScrollAnimator.cpp
index 9def297..428a79d 100644
--- a/Source/WebCore/platform/ScrollAnimator.cpp
+++ b/Source/WebCore/platform/ScrollAnimator.cpp
@@ -71,9 +71,11 @@ bool ScrollAnimator::scroll(ScrollbarOrientation orientation, ScrollGranularity,
 
 void ScrollAnimator::scrollToOffsetWithoutAnimation(const FloatPoint& offset)
 {
-    m_currentPosX = offset.x();
-    m_currentPosY = offset.y();
-    notityPositionChanged();
+    if (m_currentPosX != offset.x() || m_currentPosY != offset.y()) {
+        m_currentPosX = offset.x();
+        m_currentPosY = offset.y();
+        notityPositionChanged();
+    }
 }
 
 FloatPoint ScrollAnimator::currentPosition() const

-- 
WebKit Debian packaging



More information about the Pkg-webkit-commits mailing list