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

Unified Diff: third_party/WebKit/Source/core/layout/svg/SVGLayoutTreeAsText.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/SVGLayoutTreeAsText.cpp
diff --git a/third_party/WebKit/Source/core/layout/svg/SVGLayoutTreeAsText.cpp b/third_party/WebKit/Source/core/layout/svg/SVGLayoutTreeAsText.cpp
index 9a9b466ea600a1a755a93ce99897eb79ac2bc4de..e781b0752fc9eb421c6d573278b3a63d95b9c4c2 100644
--- a/third_party/WebKit/Source/core/layout/svg/SVGLayoutTreeAsText.cpp
+++ b/third_party/WebKit/Source/core/layout/svg/SVGLayoutTreeAsText.cpp
@@ -71,7 +71,8 @@
namespace blink {
-/** class + iomanip to help streaming list separators, i.e. ", " in string "a, b, c, d"
+/** class + iomanip to help streaming list separators, i.e. ", " in string "a,
+ * b, c, d"
* Can be used in cases where you don't know which item in the list is the first
* one to be printed, but still want to avoid strings like ", b, c".
*/
@@ -404,7 +405,8 @@ static void writeLayoutSVGTextBox(TextStream& ts, const LayoutSVGText& text) {
LayoutPoint(text.location()),
LayoutSize(box->logicalWidth(), box->logicalHeight())));
- // FIXME: Remove this hack, once the new text layout engine is completly landed. We want to preserve the old layout test results for now.
+ // FIXME: Remove this hack, once the new text layout engine is completly
+ // landed. We want to preserve the old layout test results for now.
ts << " contains 1 chunk(s)";
if (text.parent() && (text.parent()->resolveColor(CSSPropertyColor) !=
@@ -435,7 +437,8 @@ static inline void writeSVGInlineTextBox(TextStream& ts,
unsigned startOffset = fragment.characterOffset;
unsigned endOffset = fragment.characterOffset + fragment.length;
- // FIXME: Remove this hack, once the new text layout engine is completly landed. We want to preserve the old layout test results for now.
+ // FIXME: Remove this hack, once the new text layout engine is completly
+ // landed. We want to preserve the old layout test results for now.
ts << "chunk 1 ";
ETextAnchor anchor = svgStyle.textAnchor();
bool isVerticalText = !textLineLayout.style()->isHorizontalWritingMode();
@@ -569,8 +572,10 @@ void writeSVGResourceContainer(TextStream& ts,
LayoutSVGResourcePattern* pattern =
static_cast<LayoutSVGResourcePattern*>(resource);
- // Dump final results that are used for layout. No use in asking SVGPatternElement for its patternUnits(), as it may
- // link to other patterns using xlink:href, we need to build the full inheritance chain, aka. collectPatternProperties()
+ // Dump final results that are used for layout. No use in asking
+ // SVGPatternElement for its patternUnits(), as it may link to other
+ // patterns using xlink:href, we need to build the full inheritance chain,
+ // aka. collectPatternProperties()
PatternAttributes attributes;
toSVGPatternElement(pattern->element())
->collectPatternAttributes(attributes);
@@ -587,8 +592,10 @@ void writeSVGResourceContainer(TextStream& ts,
LayoutSVGResourceLinearGradient* gradient =
static_cast<LayoutSVGResourceLinearGradient*>(resource);
- // Dump final results that are used for layout. No use in asking SVGGradientElement for its gradientUnits(), as it may
- // link to other gradients using xlink:href, we need to build the full inheritance chain, aka. collectGradientProperties()
+ // Dump final results that are used for layout. No use in asking
+ // SVGGradientElement for its gradientUnits(), as it may link to other
+ // gradients using xlink:href, we need to build the full inheritance chain,
+ // aka. collectGradientProperties()
LinearGradientAttributes attributes;
toSVGLinearGradientElement(gradient->element())
->collectGradientAttributes(attributes);
@@ -602,8 +609,10 @@ void writeSVGResourceContainer(TextStream& ts,
LayoutSVGResourceRadialGradient* gradient =
toLayoutSVGResourceRadialGradient(resource);
- // Dump final results that are used for layout. No use in asking SVGGradientElement for its gradientUnits(), as it may
- // link to other gradients using xlink:href, we need to build the full inheritance chain, aka. collectGradientProperties()
+ // Dump final results that are used for layout. No use in asking
+ // SVGGradientElement for its gradientUnits(), as it may link to other
+ // gradients using xlink:href, we need to build the full inheritance chain,
+ // aka. collectGradientProperties()
RadialGradientAttributes attributes;
toSVGRadialGradientElement(gradient->element())
->collectGradientAttributes(attributes);
@@ -692,8 +701,10 @@ void writeResources(TextStream& ts, const LayoutObject& object, int indent) {
const ComputedStyle& style = object.styleRef();
const SVGComputedStyle& svgStyle = style.svgStyle();
- // FIXME: We want to use SVGResourcesCache to determine which resources are present, instead of quering the resource <-> id cache.
- // For now leave the DRT output as is, but later on we should change this so cycles are properly ignored in the DRT output.
+ // FIXME: We want to use SVGResourcesCache to determine which resources are
+ // present, instead of quering the resource <-> id cache.
+ // For now leave the DRT output as is, but later on we should change this so
+ // cycles are properly ignored in the DRT output.
if (!svgStyle.maskerResource().isEmpty()) {
if (LayoutSVGResourceMasker* masker =
getLayoutSVGResourceById<LayoutSVGResourceMasker>(

Powered by Google App Engine
This is Rietveld 408576698