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

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

Issue 2391693004: Reformat comments in core/layout/svg/line (Closed)
Patch Set: Rebase w/HEAD 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
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/layout/svg/line/SVGRootInlineBox.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/core/layout/svg/line/SVGInlineTextBox.cpp
diff --git a/third_party/WebKit/Source/core/layout/svg/line/SVGInlineTextBox.cpp b/third_party/WebKit/Source/core/layout/svg/line/SVGInlineTextBox.cpp
index 06e06db5f96ad18ea82f460769034878c3a01a39..966ec7c7a08a1de89f612f9f558f4c7744f5f51a 100644
--- a/third_party/WebKit/Source/core/layout/svg/line/SVGInlineTextBox.cpp
+++ b/third_party/WebKit/Source/core/layout/svg/line/SVGInlineTextBox.cpp
@@ -1,4 +1,4 @@
-/**
+/*
* Copyright (C) 2007 Rob Buis <buis@kde.org>
* Copyright (C) 2007 Nikolas Zimmermann <zimmermann@kde.org>
* Copyright (C) Research In Motion Limited 2010. All rights reserved.
@@ -47,19 +47,20 @@ SVGInlineTextBox::SVGInlineTextBox(LineLayoutItem item,
void SVGInlineTextBox::dirtyLineBoxes() {
InlineTextBox::dirtyLineBoxes();
- // Clear the now stale text fragments
+ // Clear the now stale text fragments.
clearTextFragments();
- // And clear any following text fragments as the text on which they
- // depend may now no longer exist, or glyph positions may be wrong
+ // And clear any following text fragments as the text on which they depend may
+ // now no longer exist, or glyph positions may be wrong.
InlineTextBox* nextBox = nextTextBox();
if (nextBox)
nextBox->dirtyLineBoxes();
}
int SVGInlineTextBox::offsetForPosition(LayoutUnit, bool) const {
- // SVG doesn't use the standard offset <-> position selection system, as it's not suitable for SVGs complex needs.
- // vertical text selection, inline boxes spanning multiple lines (contrary to HTML, etc.)
+ // SVG doesn't use the standard offset <-> position selection system, as it's
+ // not suitable for SVGs complex needs. Vertical text selection, inline boxes
+ // spanning multiple lines (contrary to HTML, etc.)
ASSERT_NOT_REACHED();
return 0;
}
@@ -198,7 +199,8 @@ TextRun SVGInlineTextBox::constructTextRun(
// 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() - fragment.characterOffset);
ASSERT(run.charactersLength() >= run.length());
return run;
@@ -228,7 +230,8 @@ void SVGInlineTextBox::paintDocumentMarker(GraphicsContext&,
const ComputedStyle&,
const Font&,
bool) const {
- // SVG does not have support for generic document markers (e.g., spellchecking, etc).
+ // SVG does not have support for generic document markers (e.g.,
+ // spellchecking, etc).
}
void SVGInlineTextBox::paintTextMatchMarkerForeground(
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/layout/svg/line/SVGRootInlineBox.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698