Commit a729edf9 authored by eae's avatar eae Committed by Commit bot

Reformat comments in core/layout/svg/line

BUG=563793
R=pdr@chromium.org

Review-Url: https://codereview.chromium.org/2391693004
Cr-Commit-Position: refs/heads/master@{#423670}
parent 9e74eb9a
/**
/*
* 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(
......
......@@ -67,7 +67,8 @@ void SVGRootInlineBox::computePerCharacterLayoutInformation() {
characterLayout.finishLayout();
// Perform SVG text layout phase four
// Position & resize all SVGInlineText/FlowBoxes in the inline box tree, resize the root box as well as the LayoutSVGText parent block.
// Position & resize all SVGInlineText/FlowBoxes in the inline box tree,
// resize the root box as well as the LayoutSVGText parent block.
LayoutRect childRect;
layoutChildBoxes(this, &childRect);
layoutRootBox(childRect);
......@@ -192,7 +193,8 @@ static inline void reverseInlineBoxRangeAndValueListsIfNeeded(
SVGInlineTextBox* firstTextBox = toSVGInlineTextBox(*first);
SVGInlineTextBox* lastTextBox = toSVGInlineTextBox(*last);
// Reordering is only necessary for BiDi text that is _absolutely_ positioned.
// Reordering is only necessary for BiDi text that is _absolutely_
// positioned.
if (firstTextBox->len() == 1 && firstTextBox->len() == lastTextBox->len())
swapPositioningValuesInTextBoxes(firstTextBox, lastTextBox);
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment