Changeset 175934 in webkit

Timestamp:

Nov 11, 2014, 8:36:57 AM (12 years ago)

Author:

Carlos Garcia Campos

Message:

Merge r175395 - 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):

Location:

releases/WebKitGTK/webkit-2.6/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• rendering/RenderObject.cpp (modified) (1 diff)

releases/WebKitGTK/webkit-2.6/Source/WebCore/ChangeLog

@@ +1 @@
+2014-10-30  Simon Fraser  <simon.fraser@apple.com>
+
+        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-24  Philippe Normand  <pnormand@igalia.com>
 

releases/WebKitGTK/webkit-2.6/Source/WebCore/rendering/RenderObject.cpp

@@ -1232 +1232 @@
 void RenderObject::repaintUsingContainer(const RenderLayerModelObject* repaintContainer, const LayoutRect& r, bool shouldClipToLayer) const
 {
+    if (r.isEmpty())
+        return;
+
     if (!repaintContainer) {
         view().repaintViewRectangle(r);