Title: [109003] trunk/Source/WebCore
Revision
109003
Author
commit-qu...@webkit.org
Date
2012-02-27 09:57:55 -0800 (Mon, 27 Feb 2012)

Log Message

Stop recomputing SVG path data twice during layout
https://bugs.webkit.org/show_bug.cgi?id=79672

Patch by Philip Rogers <p...@google.com> on 2012-02-27
Reviewed by Nikolas Zimmermann.

* rendering/svg/RenderSVGShape.cpp:
(WebCore::RenderSVGShape::layout):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (109002 => 109003)


--- trunk/Source/WebCore/ChangeLog	2012-02-27 17:51:06 UTC (rev 109002)
+++ trunk/Source/WebCore/ChangeLog	2012-02-27 17:57:55 UTC (rev 109003)
@@ -1,3 +1,13 @@
+2012-02-27  Philip Rogers  <p...@google.com>
+
+        Stop recomputing SVG path data twice during layout
+        https://bugs.webkit.org/show_bug.cgi?id=79672
+
+        Reviewed by Nikolas Zimmermann.
+
+        * rendering/svg/RenderSVGShape.cpp:
+        (WebCore::RenderSVGShape::layout):
+
 2012-02-27  Timothy Hatcher  <timo...@apple.com>
 
         Updated for WebKit2 string changes.

Modified: trunk/Source/WebCore/rendering/svg/RenderSVGShape.cpp (109002 => 109003)


--- trunk/Source/WebCore/rendering/svg/RenderSVGShape.cpp	2012-02-27 17:51:06 UTC (rev 109002)
+++ trunk/Source/WebCore/rendering/svg/RenderSVGShape.cpp	2012-02-27 17:57:55 UTC (rev 109003)
@@ -168,7 +168,7 @@
     bool updateCachedBoundariesInParents = false;
 
     bool needsShapeUpdate = m_needsShapeUpdate;
-    if (needsShapeUpdate) {
+    if (needsShapeUpdate || m_needsBoundariesUpdate) {
         setIsPaintingFallback(false);
         m_path.clear();
         createShape();
@@ -182,13 +182,6 @@
         updateCachedBoundariesInParents = true;
     }
 
-    if (m_needsBoundariesUpdate) {
-        setIsPaintingFallback(false);
-        m_path.clear();
-        createShape();
-        updateCachedBoundariesInParents = true;
-    }
-
     // Invalidate all resources of this client if our layout changed.
     if (everHadLayout() && selfNeedsLayout()) {
         SVGResourcesCache::clientLayoutChanged(this);
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to