Changeset 139312 in webkit

Timestamp:

Jan 10, 2013 5:37:27 AM (11 years ago)

Author:

mikhail.pozdnyakov@intel.com

Message:

[EFL][WK2] Contents gets blurry when page is scrolled down
​https://bugs.webkit.org/show_bug.cgi?id=106545

Reviewed by Kenneth Rohde Christiansen.

The bound values for scrolling also should be pixel aligned, otherwise
it leads to page contents blurriness when the page is scrolled down.

• UIProcess/PageViewportController.cpp:

(WebKit::PageViewportController::pageDidRequestScroll):

Location:

trunk/Source/WebKit2

Files:

• ChangeLog (modified) (1 diff)
• UIProcess/PageViewportController.cpp (modified) (1 diff)

trunk/Source/WebKit2/ChangeLog

@@ +1 @@
+2013-01-10  Mikhail Pozdnyakov  <mikhail.pozdnyakov@intel.com>
+
+        [EFL][WK2] Contents gets blurry when page is scrolled down
+        https://bugs.webkit.org/show_bug.cgi?id=106545
+
+        Reviewed by Kenneth Rohde Christiansen.
+
+        The bound values for scrolling also should be pixel aligned, otherwise
+        it leads to page contents blurriness when the page is scrolled down.
+
+        * UIProcess/PageViewportController.cpp:
+        (WebKit::PageViewportController::pageDidRequestScroll):
+
 2013-01-10  Carlos Garcia Campos  <cgarcia@igalia.com>
 

trunk/Source/WebKit2/UIProcess/PageViewportController.cpp

@@ -229 +229 @@
         return;
 
-    FloatPoint position = pixelAlignedFloatPoint(FloatPoint(cssPosition));
-    FloatPoint boundPosition = boundContentsPosition(position);
-    FloatRect endVisibleContentRect(boundPosition, visibleContentsSize());
+    FloatPoint boundPosition = boundContentsPosition(FloatPoint(cssPosition));
+    FloatPoint alignedPosition = pixelAlignedFloatPoint(boundPosition);
+    FloatRect endVisibleContentRect(alignedPosition, visibleContentsSize());
 
     if (m_lastFrameCoveredRect.intersects(endVisibleContentRect))
-        m_client->setViewportPosition(boundPosition);
+        m_client->setViewportPosition(alignedPosition);
     else {
         // Keep the unbound position in case the contents size is changed later on.
+        FloatPoint position = pixelAlignedFloatPoint(FloatPoint(cssPosition));
         applyPositionAfterRenderingContents(position);
     }