Changeset 141770 in webkit


Ignore:
Timestamp:
Feb 4, 2013 7:00:11 AM (11 years ago)
Author:
mikhail.pozdnyakov@intel.com
Message:

[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):

Location:
trunk/Source/WebKit2
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/WebKit2/ChangeLog

    r141760 r141770  
     12013-02-04  Mikhail Pozdnyakov  <mikhail.pozdnyakov@intel.com>
     2
     3        [WK2][EFL] Weird stripe at the end of the page
     4        https://bugs.webkit.org/show_bug.cgi?id=108820
     5
     6        Reviewed by Noam Rosenthal.
     7
     8        The page scroll bound was artificially enlarged by one causing the artifact appearance.
     9
     10        * UIProcess/PageViewportController.cpp:
     11        (WebKit::PageViewportController::boundContentsPositionAtScale):
     12
    1132013-02-04  David Kilzer  <ddkilzer@apple.com>
    214
  • trunk/Source/WebKit2/UIProcess/PageViewportController.cpp

    r141310 r141770  
    126126
    127127    FloatPoint position;
    128     // Unfortunately it doesn't seem to be enough, so just always allow one pixel more.
    129     position.setX(clampTo(framePosition.x(), bounds.x(), bounds.width() + 1));
    130     position.setY(clampTo(framePosition.y(), bounds.y(), bounds.height() + 1));
     128    position.setX(clampTo(framePosition.x(), bounds.x(), bounds.width()));
     129    position.setY(clampTo(framePosition.y(), bounds.y(), bounds.height()));
    131130
    132131    return position;
Note: See TracChangeset for help on using the changeset viewer.