[webkit-reviews] review requested: [Bug 95094] [Chromium] Remove contentsScale from GraphicsLayerChromium and WebContentLayer : [Attachment 161741] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Aug 31 11:51:14 PDT 2012


Jeff Timanus <twiz at chromium.org> has asked  for review:
Bug 95094: [Chromium] Remove contentsScale from GraphicsLayerChromium and
WebContentLayer
https://bugs.webkit.org/show_bug.cgi?id=95094

Attachment 161741: Patch
https://bugs.webkit.org/attachment.cgi?id=161741&action=review

------- Additional Comments from Jeff Timanus <twiz at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=161741&action=review


> Source/WebCore/platform/graphics/chromium/cc/CCLayerTreeHost.cpp:497
> +}

I believe that this approach of applying the scale is not correct - as shown in
the screen-shot attachment.  

Can any of the reviewers comment on why this may be the case?  The only change
between this patch and the previous is that these calls were moved here,
instead of in CCLayerTreeHost::paintLayerContents &
CCLayerTreeHost::paintMasksForRenderSurface.


More information about the webkit-reviews mailing list