Reformat comments in core/layout/line

third_party/WebKit/Source/core/layout/line/InlineFlowBox.h

 /*
  * Copyright (C) 2003, 2004, 2005, 2006, 2007 Apple 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 47 matching lines @@
         m_hasAnnotationsAfter(false),
         m_lineBreakBidiStatusEor(WTF::Unicode::LeftToRight),
         m_lineBreakBidiStatusLastStrong(WTF::Unicode::LeftToRight),
         m_lineBreakBidiStatusLast(WTF::Unicode::LeftToRight),
         m_isFirstAfterPageBreak(false)
 #if ENABLE(ASSERT)
         ,
         m_hasBadChildList(false)
 #endif
   {
-    // Internet Explorer and Firefox always create a marker for list items, even when the list-style-type is none.  We do not make a marker
-    // in the list-style-type: none case, since it is wasteful to do so.  However, in order to match other browsers we have to pretend like
-    // an invisible marker exists.  The side effect of having an invisible marker is that the quirks mode behavior of shrinking lines with no
-    // text children must not apply.  This change also means that gaps will exist between image bullet list items.  Even when the list bullet
-    // is an image, the line is still considered to be immune from the quirk.
+    // Internet Explorer and Firefox always create a marker for list items, even
+    // when the list-style-type is none.  We do not make a marker in the
+    // list-style-type: none case, since it is wasteful to do so.
+    // However, in order to match other browsers we have to pretend like an
+    // invisible marker exists.  The side effect of having an invisible marker
+    // is that the quirks mode behavior of shrinking lines with no text children
+    // must not apply. This change also means that gaps will exist between image
+    // bullet list items.  Even when the list bullet is an image, the line is
+    // still considered to be immune from the quirk.
     m_hasTextChildren = lineLayoutItem.style()->display() == EDisplay::ListItem;
     m_hasTextDescendants = m_hasTextChildren;
   }
 
 #if ENABLE(ASSERT)
   ~InlineFlowBox() override;
 #endif
 
 #ifndef NDEBUG
   void showLineTreeAndMark(const InlineBox* = nullptr,
@@ skipping 190 matching lines @@
                               LayoutUnit& truncatedWidth,
                               bool&) override;
 
   bool hasTextChildren() const { return m_hasTextChildren; }
   bool hasTextDescendants() const { return m_hasTextDescendants; }
   void setHasTextDescendants() { m_hasTextDescendants = true; }
 
   void checkConsistency() const;
   void setHasBadChildList();
 
-  // Line visual and layout overflow are in the coordinate space of the block.  This means that
-  // they aren't purely physical directions. For horizontal-tb and vertical-lr they will match
-  // physical directions, but for vertical-rl, the left/right respectively are flipped when
-  // compared to their physical counterparts.  For example minX is on the left in vertical-lr, but
-  // it is on the right in vertical-rl.
+  // Line visual and layout overflow are in the coordinate space of the block.
+  // This means that they aren't purely physical directions. For horizontal-tb
+  // and vertical-lr they will match physical directions, but for vertical-rl,
+  // the left/right respectively are flipped when compared to their physical
+  // counterparts.  For example minX is on the left in vertical-lr, but it is on
+  // the right in vertical-rl.
   LayoutRect layoutOverflowRect(LayoutUnit lineTop,
                                 LayoutUnit lineBottom) const {
     return m_overflow ? m_overflow->layoutOverflowRect()
                       : frameRectIncludingLineHeight(lineTop, lineBottom);
   }
   LayoutUnit logicalTopLayoutOverflow(LayoutUnit lineTop) const {
     if (m_overflow)
       return isHorizontal() ? m_overflow->layoutOverflowRect().y()
                             : m_overflow->layoutOverflowRect().x();
     return lineTop;
@@ skipping 68 matching lines @@
     m_descendantsHaveSameLineHeightAndBaseline = false;
     if (parent() && parent()->descendantsHaveSameLineHeightAndBaseline())
       parent()->clearDescendantsHaveSameLineHeightAndBaseline();
   }
 
   bool isFirstAfterPageBreak() const { return m_isFirstAfterPageBreak; }
   void setIsFirstAfterPageBreak(bool isFirstAfterPageBreak) {
     m_isFirstAfterPageBreak = isFirstAfterPageBreak;
   }
 
-  // Some callers (LayoutListItem) needs to set extra overflow on their line box.
+  // Some callers (LayoutListItem) needs to set extra overflow on their line
+  // box.
   void overrideOverflowFromLogicalRects(const LayoutRect& logicalLayoutOverflow,
                                         const LayoutRect& logicalVisualOverflow,
                                         LayoutUnit lineTop,
                                         LayoutUnit lineBottom) {
-    // If we are setting an overflow, then we can't pretend not to have an overflow.
+    // If we are setting an overflow, then we can't pretend not to have an
+    // overflow.
     clearKnownToHaveNoOverflow();
     setOverflowFromLogicalRects(logicalLayoutOverflow, logicalVisualOverflow,
                                 lineTop, lineBottom);
   }
 
  private:
   void placeBoxRangeInInlineDirection(InlineBox* firstChild,
                                       InlineBox* lastChild,
                                       LayoutUnit& logicalLeft,
                                       LayoutUnit& minLogicalLeft,
@@ skipping 47 matching lines @@
   void computeMaxLogicalTop(LayoutUnit& maxLogicalTop) const;
 
  private:
   unsigned m_includeLogicalLeftEdge : 1;
   unsigned m_includeLogicalRightEdge : 1;
   unsigned m_hasTextChildren : 1;
   unsigned m_hasTextDescendants : 1;
   unsigned m_descendantsHaveSameLineHeightAndBaseline : 1;
 
  protected:
-  // The following members are only used by RootInlineBox but moved here to keep the bits packed.
+  // The following members are only used by RootInlineBox but moved here to keep
+  // the bits packed.
 
-  // Whether or not this line uses alphabetic or ideographic baselines by default.
+  // Whether or not this line uses alphabetic or ideographic baselines by
+  // default.
   unsigned m_baselineType : 1;  // FontBaseline
 
   // If the line contains any ruby runs, then this will be true.
   unsigned m_hasAnnotationsBefore : 1;
   unsigned m_hasAnnotationsAfter : 1;
 
   unsigned m_lineBreakBidiStatusEor : 5;         // WTF::Unicode::Direction
   unsigned m_lineBreakBidiStatusLastStrong : 5;  // WTF::Unicode::Direction
   unsigned m_lineBreakBidiStatusLast : 5;        // WTF::Unicode::Direction
 
@@ skipping 15 matching lines @@
 
 inline void InlineFlowBox::setHasBadChildList() {
 #if ENABLE(ASSERT)
   m_hasBadChildList = true;
 #endif
 }
 
 }  // namespace blink
 
 #endif  // InlineFlowBox_h