Reformat comments in core/layout up until LayoutTableRow

third_party/WebKit/Source/core/layout/LayoutTableCol.cpp

 /*
  * Copyright (C) 1997 Martin Jones (mjones@kde.org)
  *           (C) 1997 Torben Weis (weis@kde.org)
  *           (C) 1998 Waldo Bastian (bastian@kde.org)
  *           (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  * Copyright (C) 2003, 2004, 2005, 2006, 2009 Apple Inc. All rights reserved.
  * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies)
  *
  * This library is free software; you can redistribute it and/or
@@ skipping 37 matching lines @@
 
   LayoutTableBoxComponent::styleDidChange(diff, oldStyle);
 
   if (!oldStyle)
     return;
 
   LayoutTable* table = this->table();
   if (!table)
     return;
 
-  // TODO(dgrogan): Is the "else" necessary for correctness or just a brittle optimization? The optimization would be:
-  // if the first branch is taken then the next one can't be, so don't even check its condition.
+  // TODO(dgrogan): Is the "else" necessary for correctness or just a brittle
+  // optimization? The optimization would be: if the first branch is taken then
+  // the next one can't be, so don't even check its condition.
   if (!table->selfNeedsLayout() && !table->normalChildNeedsLayout() &&
       oldStyle->border() != style()->border()) {
     table->invalidateCollapsedBorders();
   } else if ((oldStyle->logicalWidth() != style()->logicalWidth()) ||
              LayoutTableBoxComponent::doCellsHaveDirtyWidth(*this, *table, diff,
                                                             *oldStyle)) {
     // TODO(dgrogan): Optimization opportunities:
-    // (1) Only mark cells which are affected by this col, not every cell in the table.
-    // (2) If only the col width changes and its border width doesn't, do the cells need to be marked as
-    //     needing layout or just given dirty widths?
+    // (1) Only mark cells which are affected by this col, not every cell in the
+    //     table.
+    // (2) If only the col width changes and its border width doesn't, do the
+    //     cells need to be marked as needing layout or just given dirty
+    //     widths?
     table->markAllCellsWidthsDirtyAndOrNeedsLayout(
         LayoutTable::MarkDirtyAndNeedsLayout);
   }
 }
 
 void LayoutTableCol::updateFromElement() {
   unsigned oldSpan = m_span;
   Node* n = node();
   if (isHTMLTableColElement(n)) {
     HTMLTableColElement& tc = toHTMLTableColElement(*n);
@@ skipping 23 matching lines @@
 }
 
 bool LayoutTableCol::canHaveChildren() const {
   // Cols cannot have children. This is actually necessary to fix a bug
   // with libraries.uc.edu, which makes a <p> be a table-column.
   return isTableColumnGroup();
 }
 
 LayoutRect LayoutTableCol::localOverflowRectForPaintInvalidation() const {
   // Entire table gets invalidated, instead of invalidating
-  // every cell in the column.
-  // This is simpler, but suboptimal.
+  // every cell in the column. This is simpler, but suboptimal.
 
   LayoutTable* table = this->table();
   if (!table)
     return LayoutRect();
 
   // The correctness of this method depends on the fact that LayoutTableCol's
   // location is always zero.
   ASSERT(this->location() == LayoutPoint());
 
   return table->localOverflowRectForPaintInvalidation();
@@ skipping 17 matching lines @@
   if (!parent()->isLayoutTableCol())
     return nullptr;
 
   LayoutTableCol* parentColumnGroup = toLayoutTableCol(parent());
   ASSERT(parentColumnGroup->isTableColumnGroup());
   ASSERT(isTableColumn());
   return parentColumnGroup;
 }
 
 LayoutTableCol* LayoutTableCol::nextColumn() const {
-  // If |this| is a column-group, the next column is the colgroup's first child column.
+  // If |this| is a column-group, the next column is the colgroup's first child
+  // column.
   if (LayoutObject* firstChild = this->firstChild())
     return toLayoutTableCol(firstChild);
 
   // Otherwise it's the next column along.
   LayoutObject* next = nextSibling();
 
-  // Failing that, the child is the last column in a column-group, so the next column is the next column/column-group after its column-group.
+  // Failing that, the child is the last column in a column-group, so the next
+  // column is the next column/column-group after its column-group.
   if (!next && parent()->isLayoutTableCol())
     next = parent()->nextSibling();
 
   for (; next && !next->isLayoutTableCol(); next = next->nextSibling()) {
   }
 
   return toLayoutTableCol(next);
 }
 
 const BorderValue& LayoutTableCol::borderAdjoiningCellStartBorder(
@@ skipping 19 matching lines @@
 const BorderValue& LayoutTableCol::borderAdjoiningCellAfter(
     const LayoutTableCell* cell) const {
   DCHECK_EQ(table()
                 ->colElementAtAbsoluteColumn(cell->absoluteColumnIndex() - 1)
                 .innermostColOrColGroup(),
             this);
   return style()->borderEnd();
 }
 
 }  // namespace blink