Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4)

Side by Side Diff: Source/core/rendering/svg/RenderSVGShape.cpp

Issue 693423002: Devirtualize animatedLocalTransform() and rename it for clarity (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Remove some codes that snuck in Created 6 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2004, 2005, 2007 Nikolas Zimmermann <zimmermann@kde.org> 2 * Copyright (C) 2004, 2005, 2007 Nikolas Zimmermann <zimmermann@kde.org>
3 * Copyright (C) 2004, 2005, 2008 Rob Buis <buis@kde.org> 3 * Copyright (C) 2004, 2005, 2008 Rob Buis <buis@kde.org>
4 * Copyright (C) 2005, 2007 Eric Seidel <eric@webkit.org> 4 * Copyright (C) 2005, 2007 Eric Seidel <eric@webkit.org>
5 * Copyright (C) 2009 Google, Inc. 5 * Copyright (C) 2009 Google, Inc.
6 * Copyright (C) 2009 Dirk Schulze <krit@webkit.org> 6 * Copyright (C) 2009 Dirk Schulze <krit@webkit.org>
7 * Copyright (C) Research In Motion Limited 2010. All rights reserved. 7 * Copyright (C) Research In Motion Limited 2010. All rights reserved.
8 * Copyright (C) 2009 Jeff Schiller <codedread@gmail.com> 8 * Copyright (C) 2009 Jeff Schiller <codedread@gmail.com>
9 * Copyright (C) 2011 Renata Hodovan <reni@webkit.org> 9 * Copyright (C) 2011 Renata Hodovan <reni@webkit.org>
10 * Copyright (C) 2011 University of Szeged 10 * Copyright (C) 2011 University of Szeged
(...skipping 105 matching lines...) Expand 10 before | Expand all | Expand 10 after
116 116
117 if (m_needsShapeUpdate || m_needsBoundariesUpdate) { 117 if (m_needsShapeUpdate || m_needsBoundariesUpdate) {
118 updateShapeFromElement(); 118 updateShapeFromElement();
119 m_needsShapeUpdate = false; 119 m_needsShapeUpdate = false;
120 updatePaintInvalidationBoundingBox(); 120 updatePaintInvalidationBoundingBox();
121 m_needsBoundariesUpdate = false; 121 m_needsBoundariesUpdate = false;
122 updateCachedBoundariesInParents = true; 122 updateCachedBoundariesInParents = true;
123 } 123 }
124 124
125 if (m_needsTransformUpdate) { 125 if (m_needsTransformUpdate) {
126 m_localTransform = toSVGGraphicsElement(element())->animatedLocalTransf orm(); 126 m_localTransform = toSVGGraphicsElement(element())->calculateAnimatedLo calTransform();
127 m_needsTransformUpdate = false; 127 m_needsTransformUpdate = false;
128 updateCachedBoundariesInParents = true; 128 updateCachedBoundariesInParents = true;
129 } 129 }
130 130
131 // Invalidate all resources of this client if our layout changed. 131 // Invalidate all resources of this client if our layout changed.
132 if (everHadLayout() && selfNeedsLayout()) 132 if (everHadLayout() && selfNeedsLayout())
133 SVGResourcesCache::clientLayoutChanged(this); 133 SVGResourcesCache::clientLayoutChanged(this);
134 134
135 // If our bounds changed, notify the parents. 135 // If our bounds changed, notify the parents.
136 if (updateCachedBoundariesInParents) 136 if (updateCachedBoundariesInParents)
(...skipping 112 matching lines...) Expand 10 before | Expand all | Expand 10 after
249 bool RenderSVGShape::hasSmoothStroke() const 249 bool RenderSVGShape::hasSmoothStroke() const
250 { 250 {
251 const SVGRenderStyle& svgStyle = style()->svgStyle(); 251 const SVGRenderStyle& svgStyle = style()->svgStyle();
252 return svgStyle.strokeDashArray()->isEmpty() 252 return svgStyle.strokeDashArray()->isEmpty()
253 && svgStyle.strokeMiterLimit() == SVGRenderStyle::initialStrokeMiterLimi t() 253 && svgStyle.strokeMiterLimit() == SVGRenderStyle::initialStrokeMiterLimi t()
254 && svgStyle.joinStyle() == SVGRenderStyle::initialJoinStyle() 254 && svgStyle.joinStyle() == SVGRenderStyle::initialJoinStyle()
255 && svgStyle.capStyle() == SVGRenderStyle::initialCapStyle(); 255 && svgStyle.capStyle() == SVGRenderStyle::initialCapStyle();
256 } 256 }
257 257
258 } 258 }
OLDNEW
« no previous file with comments | « Source/core/rendering/svg/RenderSVGResourceClipper.cpp ('k') | Source/core/rendering/svg/RenderSVGText.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698