Don't apply page scale to clipRect if applyPageScaleFactorInCompositor is set to true
Created attachment 191060 [details] Patch
Comment on attachment 191060 [details] Patch CQ+ per Min's request.
Comment on attachment 191060 [details] Patch Clearing flags on attachment: 191060 Committed r144642: <http://trac.webkit.org/changeset/144642>
All reviewed patches have been landed. Closing bug.
Comment on attachment 191060 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=191060&action=review > Source/WebCore/rendering/RenderLayerCompositor.cpp:787 > + if (Settings* settings = m_renderView->document()->settings()) > + if (!settings->applyPageScaleFactorInCompositor()) > + clipRect.scale(pageScaleFactor()); WebKit coding style requires braces around the body of the outer if since the inner if is two lines long.
(In reply to comment #5) > (From update of attachment 191060 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=191060&action=review > > > Source/WebCore/rendering/RenderLayerCompositor.cpp:787 > > + if (Settings* settings = m_renderView->document()->settings()) > > + if (!settings->applyPageScaleFactorInCompositor()) > > + clipRect.scale(pageScaleFactor()); > > WebKit coding style requires braces around the body of the outer if since the inner if is two lines long. hmm... this file seems to have the same style error in another place. will fix both in a separate patch.