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

Unified Diff: third_party/WebKit/Source/core/layout/svg/SVGResources.cpp

Issue 2400783002: Reformat comments in core/layout/svg (Closed)
Patch Set: 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/layout/svg/SVGResources.cpp
diff --git a/third_party/WebKit/Source/core/layout/svg/SVGResources.cpp b/third_party/WebKit/Source/core/layout/svg/SVGResources.cpp
index 930d0312ce86c2a3a6d18526eefd8eeec617fafb..06642e4b669c62849bba527b91cfd2987378cba8 100644
--- a/third_party/WebKit/Source/core/layout/svg/SVGResources.cpp
+++ b/third_party/WebKit/Source/core/layout/svg/SVGResources.cpp
@@ -47,27 +47,33 @@ static HashSet<AtomicString>& clipperFilterMaskerTags() {
DEFINE_STATIC_LOCAL(
HashSet<AtomicString>, s_tagList,
({
- // "container elements": http://www.w3.org/TR/SVG11/intro.html#TermContainerElement
- // "graphics elements" : http://www.w3.org/TR/SVG11/intro.html#TermGraphicsElement
+ // "container elements":
+ // http://www.w3.org/TR/SVG11/intro.html#TermContainerElement
+ // "graphics elements" :
+ // http://www.w3.org/TR/SVG11/intro.html#TermGraphicsElement
aTag.localName(), circleTag.localName(), ellipseTag.localName(),
gTag.localName(), imageTag.localName(), lineTag.localName(),
markerTag.localName(), maskTag.localName(), pathTag.localName(),
polygonTag.localName(), polylineTag.localName(), rectTag.localName(),
svgTag.localName(), textTag.localName(), useTag.localName(),
- // Not listed in the definitions is the clipPath element, the SVG spec says though:
- // The "clipPath" element or any of its children can specify property "clip-path".
- // So we have to add clipPathTag here, otherwhise clip-path on clipPath will fail.
- // (Already mailed SVG WG, waiting for a solution)
+ // Not listed in the definitions is the clipPath element, the SVG spec
+ // says though:
+ // The "clipPath" element or any of its children can specify property
+ // "clip-path".
+ // So we have to add clipPathTag here, otherwhise clip-path on
+ // clipPath will fail. (Already mailed SVG WG, waiting for a solution)
clipPathTag.localName(),
- // Not listed in the definitions are the text content elements, though filter/clipper/masker on tspan/text/.. is allowed.
+ // Not listed in the definitions are the text content elements, though
+ // filter/clipper/masker on tspan/text/.. is allowed.
// (Already mailed SVG WG, waiting for a solution)
textPathTag.localName(), tspanTag.localName(),
- // Not listed in the definitions is the foreignObject element, but clip-path
- // is a supported attribute.
+ // Not listed in the definitions is the foreignObject element, but
+ // clip-path is a supported attribute.
foreignObjectTag.localName(),
// Elements that we ignore, as it doesn't make any sense.
// defs, pattern, switch (FIXME: Mail SVG WG about these)
- // symbol (is converted to a svg element, when referenced by use, we can safely ignore it.)
+ // symbol (is converted to a svg element, when referenced by use, we
+ // can safely ignore it.)
}));
return s_tagList;
}

Powered by Google App Engine
This is Rietveld 408576698