Title: [175395] trunk/Source/WebCore
Revision
175395
Author
[email protected]
Date
2014-10-30 16:26:17 -0700 (Thu, 30 Oct 2014)

Log Message

Short-circuit repaints with empty rects
https://bugs.webkit.org/show_bug.cgi?id=138234

Reviewed by Zalan Bujtas.

Repaints with an empty rectangle are common enough that it's worth
short-circuiting them.

* rendering/RenderObject.cpp:
(WebCore::RenderObject::repaintUsingContainer):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (175394 => 175395)


--- trunk/Source/WebCore/ChangeLog	2014-10-30 23:24:32 UTC (rev 175394)
+++ trunk/Source/WebCore/ChangeLog	2014-10-30 23:26:17 UTC (rev 175395)
@@ -1,3 +1,16 @@
+2014-10-30  Simon Fraser  <[email protected]>
+
+        Short-circuit repaints with empty rects
+        https://bugs.webkit.org/show_bug.cgi?id=138234
+
+        Reviewed by Zalan Bujtas.
+        
+        Repaints with an empty rectangle are common enough that it's worth
+        short-circuiting them.
+
+        * rendering/RenderObject.cpp:
+        (WebCore::RenderObject::repaintUsingContainer):
+
 2014-10-30  Chris Dumez  <[email protected]>
 
         Clean up virtual functions in css/

Modified: trunk/Source/WebCore/rendering/RenderObject.cpp (175394 => 175395)


--- trunk/Source/WebCore/rendering/RenderObject.cpp	2014-10-30 23:24:32 UTC (rev 175394)
+++ trunk/Source/WebCore/rendering/RenderObject.cpp	2014-10-30 23:26:17 UTC (rev 175395)
@@ -1232,6 +1232,9 @@
 
 void RenderObject::repaintUsingContainer(const RenderLayerModelObject* repaintContainer, const LayoutRect& r, bool shouldClipToLayer) const
 {
+    if (r.isEmpty())
+        return;
+
     if (!repaintContainer) {
         view().repaintViewRectangle(r);
         return;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to