Bug 210673 – Horizontal overflow overlay scrollbar is misplaced in RTL

• Status: RESOLVED
• Resolution: FIXED
• Component: Compositing

[Simon Fraser (smfr)]

Created attachment 396801 [details]
See the third box. turn layer borders on

We put the horizontal scrollbar in the wrong place.

[Simon Fraser (smfr)]

Created attachment 396802 [details]
Screenshot

[Radar WebKit Bug Importer]

<rdar://problem/61950751>

[Simon Fraser (smfr)]

Created attachment 397055 [details]
Patch

[Antti Koivisto]

Comment on attachment 397055 [details]
Patch

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

r+ but please consider the comment.

> Source/WebCore/rendering/RenderLayer.cpp:3185
> +    IntRect hBarRect, vBarRect, cornerRect, resizerRect;
> +    getOverflowControlsRects(hBarRect, vBarRect, cornerRect, resizerRect);

I'd much prefer a function that returns a struct over one that requires awkward out parameters.

[Simon Fraser (smfr)]

Created attachment 397090 [details]
Patch

[Antti Koivisto]

Comment on attachment 397055 [details]
Patch

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

> Source/WebCore/rendering/RenderLayer.cpp:3364
> +    auto cornerRect = [](const IntRect& boundsRect, IntSize cornerSize, bool placeVerticalScrollbarOnTheLeft) {
> +        if (placeVerticalScrollbarOnTheLeft) {
> +            auto bottomLeftCorner = boundsRect.minXMaxYCorner();
> +            return IntRect { { bottomLeftCorner.x(), bottomLeftCorner.y() - cornerSize.height(), }, cornerSize };
> +        }
> +        return IntRect { boundsRect.maxXMaxYCorner() - cornerSize, cornerSize };
> +    };
> +
> +    if (scrollbarsAvoidCorner)
> +        scrollCornerRect = cornerRect(overflowControlsPositioningRect, cornerSize, placeVerticalScrollbarOnTheLeft);
> +    else
> +        scrollCornerRect = { };

You could include 'if (!scrollbarsAvoidCorner)' branch into the lambda as well. 
Not sure this single use lambda really needs arguments at all, it could just capture the variables: [&] {

Other branches in this function would probably benefit from similar use of lambdas.

[Simon Fraser (smfr)]

https://trac.webkit.org/changeset/260445/webkit