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

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

Issue 669153002: Relocate markForLayoutAndParentResourceInvalidation (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Drop boundsChanged. Created 6 years, 2 months 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 Nikolas Zimmermann <zimmermann@kde.org> 2 * Copyright (C) 2006 Nikolas Zimmermann <zimmermann@kde.org>
3 * Copyright (C) 2007 Rob Buis <buis@kde.org> 3 * Copyright (C) 2007 Rob Buis <buis@kde.org>
4 * Copyright (C) 2008 Dirk Schulze <krit@webkit.org> 4 * Copyright (C) 2008 Dirk Schulze <krit@webkit.org>
5 * Copyright (C) Research In Motion Limited 2010. All rights reserved. 5 * Copyright (C) Research In Motion Limited 2010. All rights reserved.
6 * 6 *
7 * This library is free software; you can redistribute it and/or 7 * This library is free software; you can redistribute it and/or
8 * modify it under the terms of the GNU Library General Public 8 * modify it under the terms of the GNU Library General Public
9 * License as published by the Free Software Foundation; either 9 * License as published by the Free Software Foundation; either
10 * version 2 of the License, or (at your option) any later version. 10 * version 2 of the License, or (at your option) any later version.
11 * 11 *
12 * This library is distributed in the hope that it will be useful, 12 * This library is distributed in the hope that it will be useful,
13 * but WITHOUT ANY WARRANTY; without even the implied warranty of 13 * but WITHOUT ANY WARRANTY; without even the implied warranty of
14 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU 14 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
15 * Library General Public License for more details. 15 * Library General Public License for more details.
16 * 16 *
17 * You should have received a copy of the GNU Library General Public License 17 * You should have received a copy of the GNU Library General Public License
18 * along with this library; see the file COPYING.LIB. If not, write to 18 * along with this library; see the file COPYING.LIB. If not, write to
19 * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor, 19 * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
20 * Boston, MA 02110-1301, USA. 20 * Boston, MA 02110-1301, USA.
21 */ 21 */
22 22
23 #include "config.h" 23 #include "config.h"
24 #include "core/rendering/svg/RenderSVGResource.h" 24 #include "core/rendering/svg/RenderSVGResource.h"
25 25
26 #include "core/rendering/svg/RenderSVGResourceClipper.h" 26 #include "core/rendering/style/RenderStyle.h"
27 #include "core/rendering/svg/RenderSVGResourceFilter.h" 27 #include "core/rendering/svg/RenderSVGResourceContainer.h"
28 #include "core/rendering/svg/RenderSVGResourceMasker.h"
29 #include "core/rendering/svg/SVGResources.h" 28 #include "core/rendering/svg/SVGResources.h"
30 #include "core/rendering/svg/SVGResourcesCache.h" 29 #include "core/rendering/svg/SVGResourcesCache.h"
31 #include "platform/graphics/GraphicsContext.h" 30 #include "platform/graphics/GraphicsContext.h"
32 #include "platform/graphics/GraphicsContextStateSaver.h" 31 #include "platform/graphics/GraphicsContextStateSaver.h"
33 32
34 namespace blink { 33 namespace blink {
35 34
36 SVGPaintServer::SVGPaintServer(Color color) 35 SVGPaintServer::SVGPaintServer(Color color)
37 : m_color(color) 36 : m_color(color)
38 { 37 {
(...skipping 144 matching lines...) Expand 10 before | Expand all | Expand 10 after
183 { 182 {
184 ASSERT_NOT_REACHED(); 183 ASSERT_NOT_REACHED();
185 return SVGPaintServer::invalid(); 184 return SVGPaintServer::invalid();
186 } 185 }
187 186
188 SVGPaintDescription RenderSVGResource::requestPaintDescription(const RenderObjec t& renderer, const RenderStyle* style, RenderSVGResourceMode resourceMode) 187 SVGPaintDescription RenderSVGResource::requestPaintDescription(const RenderObjec t& renderer, const RenderStyle* style, RenderSVGResourceMode resourceMode)
189 { 188 {
190 return requestPaint(renderer, style, resourceMode); 189 return requestPaint(renderer, style, resourceMode);
191 } 190 }
192 191
193 static inline void removeFromCacheAndInvalidateDependencies(RenderObject* object , bool needsLayout)
194 {
195 ASSERT(object);
196 if (SVGResources* resources = SVGResourcesCache::cachedResourcesForRenderObj ect(object)) {
197 if (RenderSVGResourceFilter* filter = resources->filter())
198 filter->removeClientFromCache(object);
199
200 if (RenderSVGResourceMasker* masker = resources->masker())
201 masker->removeClientFromCache(object);
202
203 if (RenderSVGResourceClipper* clipper = resources->clipper())
204 clipper->removeClientFromCache(object);
205 }
206
207 if (!object->node() || !object->node()->isSVGElement())
208 return;
209 SVGElementSet* dependencies = toSVGElement(object->node())->setOfIncomingRef erences();
210 if (!dependencies)
211 return;
212
213 // We allow cycles in SVGDocumentExtensions reference sets in order to avoid expensive
214 // reference graph adjustments on changes, so we need to break possible cycl es here.
215 // This strong reference is safe, as it is guaranteed that this set will be emptied
216 // at the end of recursion.
217 typedef WillBeHeapHashSet<RawPtrWillBeMember<SVGElement> > SVGElementSet;
218 DEFINE_STATIC_LOCAL(OwnPtrWillBePersistent<SVGElementSet>, invalidatingDepen dencies, (adoptPtrWillBeNoop(new SVGElementSet)));
219
220 SVGElementSet::iterator end = dependencies->end();
221 for (SVGElementSet::iterator it = dependencies->begin(); it != end; ++it) {
222 if (RenderObject* renderer = (*it)->renderer()) {
223 if (UNLIKELY(!invalidatingDependencies->add(*it).isNewEntry)) {
224 // Reference cycle: we are in process of invalidating this depen dant.
225 continue;
226 }
227
228 RenderSVGResource::markForLayoutAndParentResourceInvalidation(render er, needsLayout);
229 invalidatingDependencies->remove(*it);
230 }
231 }
232 } 192 }
233
234 void RenderSVGResource::markForLayoutAndParentResourceInvalidation(RenderObject* object, bool needsLayout)
235 {
236 ASSERT(object);
237 ASSERT(object->node());
238
239 if (needsLayout && !object->documentBeingDestroyed())
240 object->setNeedsLayoutAndFullPaintInvalidation();
241
242 removeFromCacheAndInvalidateDependencies(object, needsLayout);
243
244 // Invalidate resources in ancestor chain, if needed.
245 RenderObject* current = object->parent();
246 while (current) {
247 removeFromCacheAndInvalidateDependencies(current, needsLayout);
248
249 if (current->isSVGResourceContainer()) {
250 // This will process the rest of the ancestors.
251 toRenderSVGResourceContainer(current)->removeAllClientsFromCache();
252 break;
253 }
254
255 current = current->parent();
256 }
257 }
258
259 }
OLDNEW
« no previous file with comments | « Source/core/rendering/svg/RenderSVGResource.h ('k') | Source/core/rendering/svg/RenderSVGResourceContainer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698