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