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

Side by Side Diff: Source/core/rendering/svg/RenderSVGImage.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) 2006 Alexander Kellett <lypanov@kde.org> 2 * Copyright (C) 2006 Alexander Kellett <lypanov@kde.org>
3 * Copyright (C) 2006 Apple Computer, Inc. 3 * Copyright (C) 2006 Apple Computer, Inc.
4 * Copyright (C) 2007 Nikolas Zimmermann <zimmermann@kde.org> 4 * Copyright (C) 2007 Nikolas Zimmermann <zimmermann@kde.org>
5 * Copyright (C) 2007, 2008, 2009 Rob Buis <buis@kde.org> 5 * Copyright (C) 2007, 2008, 2009 Rob Buis <buis@kde.org>
6 * Copyright (C) 2009 Google, Inc. 6 * Copyright (C) 2009 Google, Inc.
7 * Copyright (C) 2009 Dirk Schulze <krit@webkit.org> 7 * Copyright (C) 2009 Dirk Schulze <krit@webkit.org>
8 * Copyright (C) 2010 Patrick Gansterer <paroga@paroga.com> 8 * Copyright (C) 2010 Patrick Gansterer <paroga@paroga.com>
9 * 9 *
10 * This library is free software; you can redistribute it and/or 10 * This library is free software; you can redistribute it and/or
(...skipping 105 matching lines...) Expand 10 before | Expand all | Expand 10 after
116 } 116 }
117 117
118 void RenderSVGImage::layout() 118 void RenderSVGImage::layout()
119 { 119 {
120 ASSERT(needsLayout()); 120 ASSERT(needsLayout());
121 121
122 updateImageViewport(); 122 updateImageViewport();
123 123
124 bool transformOrBoundariesUpdate = m_needsTransformUpdate || m_needsBoundari esUpdate; 124 bool transformOrBoundariesUpdate = m_needsTransformUpdate || m_needsBoundari esUpdate;
125 if (m_needsTransformUpdate) { 125 if (m_needsTransformUpdate) {
126 m_localTransform = toSVGImageElement(element())->animatedLocalTransform( ); 126 m_localTransform = toSVGImageElement(element())->calculateAnimatedLocalT ransform();
127 m_needsTransformUpdate = false; 127 m_needsTransformUpdate = false;
128 } 128 }
129 129
130 if (m_needsBoundariesUpdate) { 130 if (m_needsBoundariesUpdate) {
131 m_bufferedForeground.clear(); 131 m_bufferedForeground.clear();
132 132
133 m_paintInvalidationBoundingBox = m_objectBoundingBox; 133 m_paintInvalidationBoundingBox = m_objectBoundingBox;
134 SVGRenderSupport::intersectPaintInvalidationRectWithResources(this, m_pa intInvalidationBoundingBox); 134 SVGRenderSupport::intersectPaintInvalidationRectWithResources(this, m_pa intInvalidationBoundingBox);
135 135
136 m_needsBoundariesUpdate = false; 136 m_needsBoundariesUpdate = false;
(...skipping 61 matching lines...) Expand 10 before | Expand all | Expand 10 after
198 198
199 void RenderSVGImage::addFocusRingRects(Vector<LayoutRect>& rects, const LayoutPo int&, const RenderLayerModelObject*) const 199 void RenderSVGImage::addFocusRingRects(Vector<LayoutRect>& rects, const LayoutPo int&, const RenderLayerModelObject*) const
200 { 200 {
201 // this is called from paint() after the localTransform has already been app lied 201 // this is called from paint() after the localTransform has already been app lied
202 LayoutRect contentRect = LayoutRect(paintInvalidationRectInLocalCoordinates( )); 202 LayoutRect contentRect = LayoutRect(paintInvalidationRectInLocalCoordinates( ));
203 if (!contentRect.isEmpty()) 203 if (!contentRect.isEmpty())
204 rects.append(contentRect); 204 rects.append(contentRect);
205 } 205 }
206 206
207 } // namespace blink 207 } // namespace blink
OLDNEW
« no previous file with comments | « Source/core/rendering/svg/RenderSVGForeignObject.cpp ('k') | Source/core/rendering/svg/RenderSVGResourceClipper.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698