Bug 64658 – Add code to attempt to align compositing layers to pixel boundaries when page scale changes

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Simon Fraser (smfr)]

Add code to attempt to align compositing layers to pixel boundaries when page scale changes

[Simon Fraser (smfr)]

Created attachment 101102 [details]
Patch

Comment on attachment 101102 [details]
Patch

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

> Source/WebCore/platform/graphics/GraphicsLayer.h:354
> +    virtual bool keepPixelAligned() const { return m_keepPixelAligned; }

This name makes it sounds as if calling this function keeps the GraphicsLayer pixel-aligned. Note how the name differs from “appliesPageScale” in this regard. I suggest “maintainsPixelAlignment” or “snapsToDevicePixels” and renaming the setter and member variable to match.

> Source/WebCore/platform/graphics/GraphicsLayerClient.h:67
>      // Multiplier for backing store size, related to high DPI.

LOL “related to high DPI”.

> Source/WebCore/platform/graphics/ca/GraphicsLayerCA.cpp:802
> +            if (currLayer->client())
> +                pageScale = currLayer->client()->pageScaleFactor();

Would it make sense to add GraphicsLayer::pageScaleFactor() that does this?

> Source/WebCore/platform/graphics/ca/GraphicsLayerCA.cpp:819
> +    float pageScaleFactor = 1;

No need to initialize this.

> Source/WebCore/platform/graphics/ca/GraphicsLayerCA.cpp:830
> +        pageScaleFactor = m_client ? m_client->pageScaleFactor() : 1;

Would it make sense to add GraphicsLayer::pageScaleFactor() that does this?

> Source/WebCore/platform/graphics/ca/GraphicsLayerCA.cpp:2439
> +    FloatRect alignedBounds = enclosingIntRect(scaledBounds);

Why always expand? I am afraid that this will turn abutting layers into overlapping layers.

[Simon Fraser (smfr)]

Created attachment 101107 [details]
Patch

[Simon Fraser (smfr)]

(In reply to comment #2)
> (From update of attachment 101102 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=101102&action=review
> 
> > Source/WebCore/platform/graphics/ca/GraphicsLayerCA.cpp:2439
> > +    FloatRect alignedBounds = enclosingIntRect(scaledBounds);
> 
> Why always expand? I am afraid that this will turn abutting layers into overlapping layers.

The added parts of the layers will always be transparent, so new overlap should not matter.

New patch ensures that we only set the 'applies scale' bit for the RenderView in main frame.

[Simon Fraser (smfr)]

Created attachment 101109 [details]
Patch

[Simon Fraser (smfr)]

http://trac.webkit.org/changeset/91161