Title: [148521] trunk/Source/WebCore
Revision
148521
Author
[email protected]
Date
2013-04-16 10:18:38 -0700 (Tue, 16 Apr 2013)

Log Message

RenderView should bail out of paintBoxDecorations() when painting with a different renderer
https://bugs.webkit.org/show_bug.cgi?id=114665
<rdar://problem/13434884>

Reviewed by Simon Fraser.

* rendering/RenderView.cpp:
(WebCore::RenderView::paintBoxDecorations): Add a check to see if we should paint within the renderer.
This check exists in all other implementations of paintBoxDecorations(), and correctly avoids painting the
decorations if the root provided is not the RenderView.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (148520 => 148521)


--- trunk/Source/WebCore/ChangeLog	2013-04-16 17:18:08 UTC (rev 148520)
+++ trunk/Source/WebCore/ChangeLog	2013-04-16 17:18:38 UTC (rev 148521)
@@ -1,3 +1,16 @@
+2013-04-15  Jon Lee  <[email protected]>
+
+        RenderView should bail out of paintBoxDecorations() when painting with a different renderer
+        https://bugs.webkit.org/show_bug.cgi?id=114665
+        <rdar://problem/13434884>
+
+        Reviewed by Simon Fraser.
+
+        * rendering/RenderView.cpp:
+        (WebCore::RenderView::paintBoxDecorations): Add a check to see if we should paint within the renderer.
+        This check exists in all other implementations of paintBoxDecorations(), and correctly avoids painting the
+        decorations if the root provided is not the RenderView.
+
 2013-04-16  Max Vujovic  <[email protected]>
 
         [CSS Shaders] Remove the cache of validated programs

Modified: trunk/Source/WebCore/rendering/RenderView.cpp (148520 => 148521)


--- trunk/Source/WebCore/rendering/RenderView.cpp	2013-04-16 17:18:08 UTC (rev 148520)
+++ trunk/Source/WebCore/rendering/RenderView.cpp	2013-04-16 17:18:38 UTC (rev 148521)
@@ -426,6 +426,9 @@
 
 void RenderView::paintBoxDecorations(PaintInfo& paintInfo, const LayoutPoint&)
 {
+    if (!paintInfo.shouldPaintWithinRoot(this))
+        return;
+
     // Check to see if we are enclosed by a layer that requires complex painting rules.  If so, we cannot blit
     // when scrolling, and we need to use slow repaints.  Examples of layers that require this are transparent layers,
     // layers with reflections, or transformed layers.
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to