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

Unified Diff: third_party/WebKit/Source/core/layout/svg/LayoutSVGInlineText.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/LayoutSVGInlineText.cpp
diff --git a/third_party/WebKit/Source/core/layout/svg/LayoutSVGInlineText.cpp b/third_party/WebKit/Source/core/layout/svg/LayoutSVGInlineText.cpp
index 052ba1e3cecad2ad5225cfa298de005fd23712bd..126a293f9ef811ae78616a0eb1e1551819ccb34a 100644
--- a/third_party/WebKit/Source/core/layout/svg/LayoutSVGInlineText.cpp
+++ b/third_party/WebKit/Source/core/layout/svg/LayoutSVGInlineText.cpp
@@ -132,7 +132,8 @@ bool LayoutSVGInlineText::characterStartsNewTextChunk(int position) const {
ASSERT(position >= 0);
ASSERT(position < static_cast<int>(textLength()));
- // Each <textPath> element starts a new text chunk, regardless of any x/y values.
+ // Each <textPath> element starts a new text chunk, regardless of any x/y
+ // values.
if (!position && parent()->isSVGTextPath() && !previousSibling())
return true;
@@ -156,7 +157,8 @@ PositionWithAffinity LayoutSVGInlineText::positionForPoint(
LayoutBlock* containingBlock = this->containingBlock();
ASSERT(containingBlock);
- // Map local point to absolute point, as the character origins stored in the text fragments use absolute coordinates.
+ // Map local point to absolute point, as the character origins stored in the
+ // text fragments use absolute coordinates.
FloatPoint absolutePoint(point);
absolutePoint.moveBy(containingBlock->location());
@@ -232,7 +234,8 @@ TextRun constructTextRun(LayoutSVGInlineText& text,
// We handle letter & word spacing ourselves.
run.disableSpacing();
- // Propagate the maximum length of the characters buffer to the TextRun, even when we're only processing a substring.
+ // Propagate the maximum length of the characters buffer to the TextRun, even
+ // when we're only processing a substring.
run.setCharactersLength(text.textLength() - position);
ASSERT(run.charactersLength() >= run.length());
return run;
@@ -364,7 +367,8 @@ void LayoutSVGInlineText::computeNewScaledFontForStyle(
ASSERT(style);
ASSERT(layoutObject);
- // Alter font-size to the right on-screen value to avoid scaling the glyphs themselves, except when GeometricPrecision is specified.
+ // Alter font-size to the right on-screen value to avoid scaling the glyphs
+ // themselves, except when GeometricPrecision is specified.
scalingFactor =
SVGLayoutSupport::calculateScreenFontSizeScalingFactor(layoutObject);
if (style->effectiveZoom() == 1 && (scalingFactor == 1 || !scalingFactor)) {
@@ -379,7 +383,8 @@ void LayoutSVGInlineText::computeNewScaledFontForStyle(
FontDescription fontDescription(style->getFontDescription());
Document& document = layoutObject->document();
- // FIXME: We need to better handle the case when we compute very small fonts below (below 1pt).
+ // FIXME: We need to better handle the case when we compute very small fonts
+ // below (below 1pt).
fontDescription.setComputedSize(FontSize::getComputedSizeFromSpecifiedSize(
&document, scalingFactor, fontDescription.isAbsoluteSize(),
fontDescription.specifiedSize(), DoNotUseSmartMinimumForFontSize));

Powered by Google App Engine
This is Rietveld 408576698