reflow comments in platform/fonts

third_party/WebKit/Source/platform/fonts/shaping/SimpleShaper.cpp

 /*
- * Copyright (C) 2003, 2006, 2008, 2009, 2010, 2011 Apple Inc. All rights reserved.
+ * Copyright (C) 2003, 2006, 2008, 2009, 2010, 2011 Apple Inc. All rights
+ * reserved.
  * Copyright (C) 2008 Holger Hans Peter Freyther
  * Copyright (C) 2014 Google Inc. 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,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 68 matching lines @@
                                    const GlyphData& glyphData) const {
   const SimpleFontData* fontData = glyphData.fontData;
   ASSERT(fontData);
 
   if (UNLIKELY(character == tabulationCharacter && m_textRun.allowTabs()))
     return m_font->tabWidth(*fontData, m_textRun.getTabSize(),
                             m_textRun.xPos() + m_runWidthSoFar);
 
   float width = fontData->widthForGlyph(glyphData.glyph);
 
-  // SVG uses horizontalGlyphStretch(), when textLength is used to stretch/squeeze text.
+  // SVG uses horizontalGlyphStretch(), when textLength is used to
+  // stretch/squeeze text.
   if (UNLIKELY(m_textRun.horizontalGlyphStretch() != 1))
     width *= m_textRun.horizontalGlyphStretch();
 
   return width;
 }
 
 float SimpleShaper::adjustSpacing(float width, const CharacterData& charData) {
   // Account for letter-spacing.
   if (width)
     width += m_font->getFontDescription().letterSpacing();
 
   bool isExpansionOpportunity =
       Character::treatAsSpace(charData.character) ||
       (m_textRun.getTextJustify() == TextJustifyDistribute);
   if (isExpansionOpportunity ||
       (m_textRun.getTextJustify() == TextJustifyAuto &&
        Character::isCJKIdeographOrSymbol(charData.character))) {
-    // Distribute the run's total expansion evenly over all expansion opportunities in the run.
+    // Distribute the run's total expansion evenly over all expansion
+    // opportunities in the run.
     if (m_expansion) {
       if (!isExpansionOpportunity && !m_isAfterExpansion) {
         // Take the expansion opportunity before this ideograph.
         m_expansion -= m_expansionPerOpportunity;
         m_runWidthSoFar += m_expansionPerOpportunity;
       }
       if (m_textRun.allowsTrailingExpansion() ||
           (m_textRun.ltr() &&
            charData.characterOffset + charData.clusterLength <
                m_textRun.length()) ||
           (m_textRun.rtl() && charData.characterOffset)) {
         m_expansion -= m_expansionPerOpportunity;
         width += m_expansionPerOpportunity;
         m_isAfterExpansion = true;
       }
     } else {
       m_isAfterExpansion = false;
     }
 
     // Account for word spacing.
-    // We apply additional space between "words" by adding width to the space character.
+    // We apply additional space between "words" by adding width to the space
+    // character.
     if (isExpansionOpportunity &&
         (charData.character != tabulationCharacter || !m_textRun.allowTabs()) &&
         (charData.characterOffset ||
          charData.character == noBreakSpaceCharacter) &&
         m_font->getFontDescription().wordSpacing()) {
       width += m_font->getFontDescription().wordSpacing();
     }
   } else {
     m_isAfterExpansion = false;
   }
@@ skipping 43 matching lines @@
     }
 
     if (hasExtraSpacing && !spaceUsedAsZeroWidthSpace)
       width = adjustSpacing(width, charData);
 
     if (m_glyphBoundingBox) {
       ASSERT(glyphData.fontData);
       FloatRect glyphBounds =
           glyphData.fontData->boundsForGlyph(glyphData.glyph);
       // We are handling simple text run here, so Y-Offset will be zero.
-      // FIXME: Computing bounds relative to the initial advance seems odd. Are we adjusting
-      // these someplace else? If not, we'll end up with different bounds depending on how
-      // we segment our advance() calls.
+      // FIXME: Computing bounds relative to the initial advance seems odd. Are
+      // we adjusting these someplace else? If not, we'll end up with different
+      // bounds depending on how we segment our advance() calls.
       glyphBounds.move(m_runWidthSoFar - initialRunWidth, 0);
       m_glyphBoundingBox->unite(glyphBounds);
     }
 
     if (glyphBuffer) {
       if (!forTextEmphasis()) {
         glyphBuffer->add(glyph, fontData, m_runWidthSoFar);
       } else if (Character::canReceiveTextEmphasis(charData.character)) {
         addEmphasisMark(glyphBuffer, m_runWidthSoFar + width / 2);
       }
@@ skipping 34 matching lines @@
   float initialWidth = m_runWidthSoFar;
 
   if (!advance(m_currentCharacter + 1))
     return false;
 
   width = m_runWidthSoFar - initialWidth;
   return true;
 }
 
 }  // namespace blink