Consider backing scale factor when setting up graphics layers
Created attachment 99524 [details] Patch
<rdar://problem/8669917>
Comment on attachment 99524 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=99524&action=review > Source/WebCore/rendering/RenderLayerBacking.cpp:111 > + ASSERT(renderer()); > + ASSERT(renderer()->document()); > + ASSERT(renderer()->document()->frame()); Should these be moved to pageContentsScale()? > Source/WebCore/rendering/RenderLayerBacking.h:195 > + float pageContentsScale() const; Why not just call this pageScaleFactor() to avoid introducing another term?
Comment on attachment 99524 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=99524&action=review >> Source/WebCore/rendering/RenderLayerBacking.cpp:111 >> + ASSERT(renderer()->document()->frame()); > > Should these be moved to pageContentsScale()? Probably not. At other call sites it may be OK for some of these things to be null. >> Source/WebCore/rendering/RenderLayerBacking.h:195 >> + float pageContentsScale() const; > > Why not just call this pageScaleFactor() to avoid introducing another term? Will do.
Created attachment 99534 [details] Patch
Committed r90293: <http://trac.webkit.org/changeset/90293>