Bug 130884 – [iOS][WK2] Adjust the tile coverage on the scrollview's edges

• Component: New Bugs
• Status: RESOLVED FIXED

[Benjamin Poulain]

[iOS][WK2] Adjust the tile coverage on the scrollview's edges

[Benjamin Poulain]

Created attachment 228020 [details]
Patch

Comment on attachment 228020 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=228020&action=review

> Source/WebCore/platform/ios/ScrollViewIOS.mm:168
> +    IntSize contentSize = contentsSize();
> +    if (futureRect.maxX() > contentSize.width())
> +        futureRect.setX(contentSize.width() - futureRect.width());
> +    if (futureRect.maxY() > contentSize.height())
> +        futureRect.setY(contentSize.height() - futureRect.height());
> +    if (futureRect.x() < 0)
> +        futureRect.setX(0);
> +    if (futureRect.y() < 0)
> +        futureRect.setY(0);
> +

Can’t this be written more concisely as intersecting futureRect with the rect { FloatPoint(), contentSize() }?

[Benjamin Poulain]

Comment on attachment 228020 [details]
Patch

Clearing flags on attachment: 228020

Committed r166394: <http://trac.webkit.org/changeset/166394>

[Benjamin Poulain]

All reviewed patches have been landed.  Closing bug.