Title: [133659] trunk/Source/WebKit2
Revision
133659
Author
commit-qu...@webkit.org
Date
2012-11-06 12:44:40 -0800 (Tue, 06 Nov 2012)

Log Message

[WK2] Wrong scroll position when going back in history
https://bugs.webkit.org/show_bug.cgi?id=101358

Patch by Yael Aharon <yael.aha...@intel.com> on 2012-11-06
Reviewed by Kenneth Rohde Christiansen.

Reset the contentsSize after a new load was committed. We should not continue
to use the contentsSize of the previous page in any calculation.

* UIProcess/PageViewportController.cpp:
(WebKit::PageViewportController::didCommitLoad):

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (133658 => 133659)


--- trunk/Source/WebKit2/ChangeLog	2012-11-06 20:43:39 UTC (rev 133658)
+++ trunk/Source/WebKit2/ChangeLog	2012-11-06 20:44:40 UTC (rev 133659)
@@ -1,3 +1,16 @@
+2012-11-06  Yael Aharon  <yael.aha...@intel.com>
+
+        [WK2] Wrong scroll position when going back in history
+        https://bugs.webkit.org/show_bug.cgi?id=101358
+
+        Reviewed by Kenneth Rohde Christiansen.
+
+        Reset the contentsSize after a new load was committed. We should not continue
+        to use the contentsSize of the previous page in any calculation.
+
+        * UIProcess/PageViewportController.cpp:
+        (WebKit::PageViewportController::didCommitLoad):
+
 2012-11-06  Christophe Dumez  <christophe.du...@intel.com>
 
         [EFL][WK2] Declare all smart callbacks in EwkViewCallbacks.h

Modified: trunk/Source/WebKit2/UIProcess/PageViewportController.cpp (133658 => 133659)


--- trunk/Source/WebKit2/UIProcess/PageViewportController.cpp	2012-11-06 20:43:39 UTC (rev 133658)
+++ trunk/Source/WebKit2/UIProcess/PageViewportController.cpp	2012-11-06 20:44:40 UTC (rev 133659)
@@ -96,6 +96,9 @@
     // Do not count the previous committed page contents as covered.
     m_lastFrameCoveredRect = FloatRect();
 
+    // Do not continue to use the content size of the previous page.
+    m_contentsSize = IntSize();
+
     // Reset the position to the top, page/history scroll requests may override this before we re-enable rendering.
     applyPositionAfterRenderingContents(FloatPoint());
 }
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to