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

Unified Diff: third_party/WebKit/Source/core/svg/SVGGElement.cpp

Issue 2390773004: reflow comments in core/svg/ (Closed)
Patch Set: comments (heh!) Created 4 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/svg/SVGGElement.cpp
diff --git a/third_party/WebKit/Source/core/svg/SVGGElement.cpp b/third_party/WebKit/Source/core/svg/SVGGElement.cpp
index abe1775faed6d4557d1c7a09fcc19d728eda1211..dbd19e438b419ce58a3be3130a7b329c713bc807 100644
--- a/third_party/WebKit/Source/core/svg/SVGGElement.cpp
+++ b/third_party/WebKit/Source/core/svg/SVGGElement.cpp
@@ -33,10 +33,11 @@ inline SVGGElement::SVGGElement(Document& document,
DEFINE_NODE_FACTORY(SVGGElement)
LayoutObject* SVGGElement::createLayoutObject(const ComputedStyle& style) {
- // SVG 1.1 testsuite explicitely uses constructs like <g display="none"><linearGradient>
- // We still have to create layoutObjects for the <g> & <linearGradient> element, though the
- // subtree may be hidden - we only want the resource layoutObjects to exist so they can be
- // referenced from somewhere else.
+ // SVG 1.1 testsuite explicitely uses constructs like
+ // <g display="none"><linearGradient>
+ // We still have to create layoutObjects for the <g> & <linearGradient>
+ // element, though the subtree may be hidden - we only want the resource
+ // layoutObjects to exist so they can be referenced from somewhere else.
if (style.display() == EDisplay::None)
return new LayoutSVGHiddenContainer(this);
@@ -44,8 +45,9 @@ LayoutObject* SVGGElement::createLayoutObject(const ComputedStyle& style) {
}
bool SVGGElement::layoutObjectIsNeeded(const ComputedStyle&) {
- // Unlike SVGElement::layoutObjectIsNeeded(), we still create layoutObjects, even if
- // display is set to 'none' - which is special to SVG <g> container elements.
+ // Unlike SVGElement::layoutObjectIsNeeded(), we still create layoutObjects,
+ // even if display is set to 'none' - which is special to SVG <g> container
+ // elements.
return parentOrShadowHostElement() &&
parentOrShadowHostElement()->isSVGElement();
}

Powered by Google App Engine
This is Rietveld 408576698