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

Side by Side Diff: Source/core/svg/SVGGElement.cpp

Issue 16896019: Replace RenderArena with PartitionAlloc. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 7 years, 6 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) 2004, 2005, 2007, 2008 Nikolas Zimmermann <zimmermann@kde.org> 2 * Copyright (C) 2004, 2005, 2007, 2008 Nikolas Zimmermann <zimmermann@kde.org>
3 * Copyright (C) 2004, 2005, 2006 Rob Buis <buis@kde.org> 3 * Copyright (C) 2004, 2005, 2006 Rob Buis <buis@kde.org>
4 * 4 *
5 * This library is free software; you can redistribute it and/or 5 * This library is free software; you can redistribute it and/or
6 * modify it under the terms of the GNU Library General Public 6 * modify it under the terms of the GNU Library General Public
7 * License as published by the Free Software Foundation; either 7 * License as published by the Free Software Foundation; either
8 * version 2 of the License, or (at your option) any later version. 8 * version 2 of the License, or (at your option) any later version.
9 * 9 *
10 * This library is distributed in the hope that it will be useful, 10 * This library is distributed in the hope that it will be useful,
(...skipping 78 matching lines...) Expand 10 before | Expand all | Expand 10 after
89 89
90 SVGElementInstance::InvalidationGuard invalidationGuard(this); 90 SVGElementInstance::InvalidationGuard invalidationGuard(this);
91 91
92 if (SVGTests::handleAttributeChange(this, attrName)) 92 if (SVGTests::handleAttributeChange(this, attrName))
93 return; 93 return;
94 94
95 if (RenderObject* renderer = this->renderer()) 95 if (RenderObject* renderer = this->renderer())
96 RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer); 96 RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer);
97 } 97 }
98 98
99 RenderObject* SVGGElement::createRenderer(RenderArena* arena, RenderStyle* style ) 99 RenderObject* SVGGElement::createRenderer(RenderStyle* style)
100 { 100 {
101 // SVG 1.1 testsuite explicitely uses constructs like <g display="none"><lin earGradient> 101 // SVG 1.1 testsuite explicitely uses constructs like <g display="none"><lin earGradient>
102 // We still have to create renderers for the <g> & <linearGradient> element, though the 102 // We still have to create renderers for the <g> & <linearGradient> element, though the
103 // subtree may be hidden - we only want the resource renderers to exist so t hey can be 103 // subtree may be hidden - we only want the resource renderers to exist so t hey can be
104 // referenced from somewhere else. 104 // referenced from somewhere else.
105 if (style->display() == NONE) 105 if (style->display() == NONE)
106 return new (arena) RenderSVGHiddenContainer(this); 106 return new RenderSVGHiddenContainer(this);
107 107
108 return new (arena) RenderSVGTransformableContainer(this); 108 return new RenderSVGTransformableContainer(this);
109 } 109 }
110 110
111 bool SVGGElement::rendererIsNeeded(const NodeRenderingContext&) 111 bool SVGGElement::rendererIsNeeded(const NodeRenderingContext&)
112 { 112 {
113 // Unlike SVGStyledElement::rendererIsNeeded(), we still create renderers, e ven if 113 // Unlike SVGStyledElement::rendererIsNeeded(), we still create renderers, e ven if
114 // display is set to 'none' - which is special to SVG <g> container elements . 114 // display is set to 'none' - which is special to SVG <g> container elements .
115 return parentOrShadowHostElement() && parentOrShadowHostElement()->isSVGElem ent(); 115 return parentOrShadowHostElement() && parentOrShadowHostElement()->isSVGElem ent();
116 } 116 }
117 117
118 } 118 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698