Reformat comments in core/layout/svg/line

third_party/WebKit/Source/core/layout/svg/line/SVGInlineTextBox.cpp

-/**
+/*
  * 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.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
  * This library is distributed in the hope that it will be useful,
@@ skipping 28 matching lines @@
               "SVGInlineTextBox has an unexpected size");
 
 SVGInlineTextBox::SVGInlineTextBox(LineLayoutItem item,
                                    int start,
                                    unsigned short length)
     : InlineTextBox(item, start, length), m_startsNewTextChunk(false) {}
 
 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;
 }
 
 int SVGInlineTextBox::offsetForPositionInFragment(
     const SVGTextFragment& fragment,
     LayoutUnit position,
     bool includePartialGlyphs) const {
   LineLayoutSVGInlineText lineLayoutItem =
       LineLayoutSVGInlineText(this->getLineLayoutItem());
@@ skipping 95 matching lines @@
                              LayoutUnit,
                              LayoutUnit) const {
   SVGInlineTextBoxPainter(*this).paint(paintInfo, paintOffset);
 }
 
 TextRun SVGInlineTextBox::constructTextRun(
     const ComputedStyle& style,
     const SVGTextFragment& fragment) const {
   LineLayoutText text = getLineLayoutItem();
 
-  // FIXME(crbug.com/264211): This should not be necessary but can occur if we
-  //                          layout during layout. Remove this when 264211 is fixed.
+  // FIXME: This should not be necessary but can occur if we layout during

[pdr., 2016/10/04 22:05:41]

I will be fixing this in https://codereview.chromium.org/2390233004 but go ahead
and land this.

+  // layout. Remove this when crbug.com/264211 is fixed.
   RELEASE_ASSERT(!text.needsLayout());
 
   TextRun run(
-      static_cast<const LChar*>(
-          nullptr)  // characters, will be set below if non-zero.
-      ,
-      0  // length, will be set below if non-zero.
-      ,
-      0  // xPos, only relevant with allowTabs=true
-      ,
-      0  // padding, only relevant for justified text, not relevant for SVG
-      ,
-      TextRun::AllowTrailingExpansion, direction(),
+      // Characters, will be set below if non-zero.
+      static_cast<const LChar*>(nullptr),
+      // Length, will be set below if non-zero.
+      0,
+      // xPos, only relevant with allowTabs=true.
+      0,
+      // Padding, only relevant for justified text, not relevant for SVG.
+      0, TextRun::AllowTrailingExpansion, direction(),
       dirOverride() ||
           style.rtlOrdering() == VisualOrder /* directionalOverride */);
 
   if (fragment.length) {
     if (text.is8Bit())
       run.setText(text.characters8() + fragment.characterOffset,
                   fragment.length);
     else
       run.setText(text.characters16() + fragment.characterOffset,
                   fragment.length);
   }
 
   // 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;
 }
 
 bool SVGInlineTextBox::mapStartEndPositionsIntoFragmentCoordinates(
     const SVGTextFragment& fragment,
     int& startPosition,
     int& endPosition) const {
   int fragmentOffsetInBox =
       static_cast<int>(fragment.characterOffset) - start();
 
   // Compute positions relative to the fragment.
   startPosition -= fragmentOffsetInBox;
   endPosition -= fragmentOffsetInBox;
 
   // Intersect with the fragment range.
   startPosition = std::max(startPosition, 0);
   endPosition = std::min(endPosition, static_cast<int>(fragment.length));
 
   return startPosition < endPosition;
 }
 
 void SVGInlineTextBox::paintDocumentMarker(GraphicsContext&,
                                            const LayoutPoint&,
                                            DocumentMarker*,
                                            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(
     const PaintInfo& paintInfo,
     const LayoutPoint& point,
     DocumentMarker* marker,
     const ComputedStyle& style,
     const Font& font) const {
   SVGInlineTextBoxPainter(*this).paintTextMatchMarkerForeground(
       paintInfo, point, marker, style, font);
@@ skipping 69 matching lines @@
               return true;
           }
         }
       }
     }
   }
   return false;
 }
 
 }  // namespace blink