Changeset 52900 in webkit

Timestamp:

Jan 6, 2010 9:30:26 PM (14 years ago)

Author:

eric@webkit.org

Message:

2009-10-18 Patrick Gansterer <​paroga@paroga.com>

Reviewed by Dirk Schulze.

Improve DOM performance in large SVG files.
Check the dirty area before painting a path, which prevents unnecessary
clipping and changes to the GraphicsContext.
​https://bugs.webkit.org/show_bug.cgi?id=30055

• rendering/RenderPath.cpp: (WebCore::RenderPath::paint):

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• rendering/RenderPath.cpp (modified) (2 diffs)

trunk/WebCore/ChangeLog

@@ +1 @@
+2009-10-18 Patrick Gansterer <paroga@paroga.com>
+
+        Reviewed by Dirk Schulze.
+
+        Improve DOM performance in large SVG files.
+        Check the dirty area before painting a path, which prevents unnecessary
+        clipping and changes to the GraphicsContext.
+        https://bugs.webkit.org/show_bug.cgi?id=30055
+
+        * rendering/RenderPath.cpp:
+        (WebCore::RenderPath::paint):
+
 2010-01-06  Kent Tamura  <tkent@chromium.org>
 

trunk/WebCore/rendering/RenderPath.cpp

@@ -215 +215 @@
     if (paintInfo.context->paintingDisabled() || style()->visibility() == HIDDEN || m_path.isEmpty())
         return;
-            
+
+    FloatRect boundingBox = repaintRectInLocalCoordinates();
+    if (!boundingBox.intersects(paintInfo.rect))
+        return;
+
     paintInfo.context->save();
     paintInfo.context->concatCTM(localToParentTransform());
@@ -221 +225 @@
     SVGResourceFilter* filter = 0;
 
-    FloatRect boundingBox = repaintRectInLocalCoordinates();
     if (paintInfo.phase == PaintPhaseForeground) {
         PaintInfo savedInfo(paintInfo);