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

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

Issue 2670643004: Rename EDisplay values with k prefix. (Closed)
Patch Set: Rebase Created 3 years, 9 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
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 20 matching lines...) Expand all
31 : SVGGraphicsElement(SVGNames::gTag, document, constructionType) {} 31 : SVGGraphicsElement(SVGNames::gTag, document, constructionType) {}
32 32
33 DEFINE_NODE_FACTORY(SVGGElement) 33 DEFINE_NODE_FACTORY(SVGGElement)
34 34
35 LayoutObject* SVGGElement::createLayoutObject(const ComputedStyle& style) { 35 LayoutObject* SVGGElement::createLayoutObject(const ComputedStyle& style) {
36 // SVG 1.1 testsuite explicitely uses constructs like 36 // SVG 1.1 testsuite explicitely uses constructs like
37 // <g display="none"><linearGradient> 37 // <g display="none"><linearGradient>
38 // We still have to create layoutObjects for the <g> & <linearGradient> 38 // We still have to create layoutObjects for the <g> & <linearGradient>
39 // element, though the subtree may be hidden - we only want the resource 39 // element, though the subtree may be hidden - we only want the resource
40 // layoutObjects to exist so they can be referenced from somewhere else. 40 // layoutObjects to exist so they can be referenced from somewhere else.
41 if (style.display() == EDisplay::None) 41 if (style.display() == EDisplay::kNone)
42 return new LayoutSVGHiddenContainer(this); 42 return new LayoutSVGHiddenContainer(this);
43 43
44 return new LayoutSVGTransformableContainer(this); 44 return new LayoutSVGTransformableContainer(this);
45 } 45 }
46 46
47 bool SVGGElement::layoutObjectIsNeeded(const ComputedStyle&) { 47 bool SVGGElement::layoutObjectIsNeeded(const ComputedStyle&) {
48 // Unlike SVGElement::layoutObjectIsNeeded(), we still create layoutObjects, 48 // Unlike SVGElement::layoutObjectIsNeeded(), we still create layoutObjects,
49 // even if display is set to 'none' - which is special to SVG <g> container 49 // even if display is set to 'none' - which is special to SVG <g> container
50 // elements. 50 // elements.
51 return isValid() && hasSVGParent(); 51 return isValid() && hasSVGParent();
52 } 52 }
53 53
54 } // namespace blink 54 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698