[SCM] WebKit Debian packaging branch, debian/experimental, updated. upstream/1.3.3-9427-gc2be6fc

darin at chromium.org darin at chromium.org
Wed Dec 22 12:14:47 UTC 2010


The following commit has been merged in the debian/experimental branch:
commit 998d5f6284579acf59c520c0b8980c78e2fa131c
Author: darin at chromium.org <darin at chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date:   Tue Aug 17 20:07:45 2010 +0000

    2010-08-17  Darin Fisher  <darin at chromium.org>
    
            Reviewed by Darin Adler.
    
            [Chromium] REGRESSION: Incorrect repaint on scrolling with position:fixed elements
            https://bugs.webkit.org/show_bug.cgi?id=42949
    
            This is a partial revert of http://trac.webkit.org/changeset/63907
    
            * page/FrameView.cpp:
            (WebCore::FrameView::scrollPositionChangedViaPlatformWidget):
            (WebCore::FrameView::scrollPositionChanged):
            * platform/ScrollView.cpp:
            (WebCore::ScrollView::valueChanged):
    
    
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@65534 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index 83126b1..ad3ea90 100644
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,18 @@
+2010-08-17  Darin Fisher  <darin at chromium.org>
+
+        Reviewed by Darin Adler.
+
+        [Chromium] REGRESSION: Incorrect repaint on scrolling with position:fixed elements
+        https://bugs.webkit.org/show_bug.cgi?id=42949
+
+        This is a partial revert of http://trac.webkit.org/changeset/63907
+
+        * page/FrameView.cpp:
+        (WebCore::FrameView::scrollPositionChangedViaPlatformWidget):
+        (WebCore::FrameView::scrollPositionChanged):
+        * platform/ScrollView.cpp:
+        (WebCore::ScrollView::valueChanged):
+
 2010-08-17  Sheriff Bot  <webkit.review.bot at gmail.com>
 
         Unreviewed, rolling out r65528.
diff --git a/WebCore/page/FrameView.cpp b/WebCore/page/FrameView.cpp
index 3aaf171..70210a4 100644
--- a/WebCore/page/FrameView.cpp
+++ b/WebCore/page/FrameView.cpp
@@ -1121,13 +1121,13 @@ void FrameView::setScrollPosition(const IntPoint& scrollPoint)
 
 void FrameView::scrollPositionChangedViaPlatformWidget()
 {
+    repaintFixedElementsAfterScrolling();
     scrollPositionChanged();
 }
 
 void FrameView::scrollPositionChanged()
 {
     frame()->eventHandler()->sendScrollEvent();
-    repaintFixedElementsAfterScrolling();
 
 #if USE(ACCELERATED_COMPOSITING)
     if (RenderView* root = m_frame->contentRenderer()) {
diff --git a/WebCore/platform/ScrollView.cpp b/WebCore/platform/ScrollView.cpp
index e2d625e..b39ddc1 100644
--- a/WebCore/platform/ScrollView.cpp
+++ b/WebCore/platform/ScrollView.cpp
@@ -292,6 +292,7 @@ void ScrollView::valueChanged(Scrollbar* scrollbar)
     if (scrollbarsSuppressed())
         return;
 
+    repaintFixedElementsAfterScrolling();
     scrollContents(scrollDelta);
 }
 

-- 
WebKit Debian packaging



More information about the Pkg-webkit-commits mailing list