[WK2][EFL] Weird stripe at the end of the page
authormikhail.pozdnyakov@intel.com <mikhail.pozdnyakov@intel.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 4 Feb 2013 15:00:11 +0000 (15:00 +0000)
committermikhail.pozdnyakov@intel.com <mikhail.pozdnyakov@intel.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 4 Feb 2013 15:00:11 +0000 (15:00 +0000)
https://bugs.webkit.org/show_bug.cgi?id=108820

Reviewed by Noam Rosenthal.

The page scroll bound was artificially enlarged by one causing the artifact appearance.

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

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@141770 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit2/ChangeLog
Source/WebKit2/UIProcess/PageViewportController.cpp

index 6b2a661..86fa11a 100644 (file)
@@ -1,3 +1,15 @@
+2013-02-04  Mikhail Pozdnyakov  <mikhail.pozdnyakov@intel.com>
+
+        [WK2][EFL] Weird stripe at the end of the page
+        https://bugs.webkit.org/show_bug.cgi?id=108820
+
+        Reviewed by Noam Rosenthal.
+
+        The page scroll bound was artificially enlarged by one causing the artifact appearance.
+
+        * UIProcess/PageViewportController.cpp:
+        (WebKit::PageViewportController::boundContentsPositionAtScale):
+
 2013-02-04  David Kilzer  <ddkilzer@apple.com>
 
         Sort WebKit2 Xcode project file
index 6fa94a3..1a9bde6 100644 (file)
@@ -125,9 +125,8 @@ FloatPoint PageViewportController::boundContentsPositionAtScale(const WebCore::F
     bounds.setHeight(std::max(0.f, m_contentsSize.height() - floorf(m_viewportSize.height() / scale)));
 
     FloatPoint position;
-    // Unfortunately it doesn't seem to be enough, so just always allow one pixel more.
-    position.setX(clampTo(framePosition.x(), bounds.x(), bounds.width() + 1));
-    position.setY(clampTo(framePosition.y(), bounds.y(), bounds.height() + 1));
+    position.setX(clampTo(framePosition.x(), bounds.x(), bounds.width()));
+    position.setY(clampTo(framePosition.y(), bounds.y(), bounds.height()));
 
     return position;
 }