Remove LayoutTable::m_currentBorder

third_party/WebKit/Source/core/layout/LayoutTable.h

 /*
  * 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, 2010 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
@@ skipping 286 matching lines @@
     LayoutTableSection* sectionBelow(const LayoutTableSection*, SkipEmptySectionsValue = DoNotSkipEmptySections) const;
 
     LayoutTableCell* cellAbove(const LayoutTableCell*) const;
     LayoutTableCell* cellBelow(const LayoutTableCell*) const;
     LayoutTableCell* cellBefore(const LayoutTableCell*) const;
     LayoutTableCell* cellAfter(const LayoutTableCell*) const;
 
     typedef Vector<CollapsedBorderValue> CollapsedBorderValues;
     void invalidateCollapsedBorders();
 
-    // TODO(wangxianzhu): This method should be moved into TablePainter.
-    const CollapsedBorderValue* currentBorderValue() const { return m_currentBorder; }
-    void setCurrentBorderValue(const CollapsedBorderValue* val) const { m_currentBorder = val; }
-
     bool hasSections() const { return m_head || m_foot || m_firstBody; }
 
     void recalcSectionsIfNeeded() const
     {
         if (m_needsSectionRecalc)
             recalcSections();
     }
 
     static LayoutTable* createAnonymousWithParent(const LayoutObject*);
     LayoutBox* createAnonymousBoxWithSameTypeAs(const LayoutObject* parent) const override
@@ skipping 56 matching lines @@
 
     void addOverflowFromChildren() override;
 
     void recalcSections() const;
     void layoutCaption(LayoutTableCaption&);
 
     void distributeExtraLogicalHeight(int extraLogicalHeight);
 
     void recalcCollapsedBordersIfNeeded();
 
-    // TODO(layout-dev): All mutables in this class (except for m_currentBorder) are lazily updated by recalcSections()
-    // which is called by various getter methods (e.g. borderBefore(), borderAfter()). They allow dirty layout even after
-    // DocumentLifecycle::LayoutClean which seems not proper. crbug.com/538236.
+    // TODO(layout-dev): All mutables in this class are lazily updated by recalcSections()
+    // which is called by various getter methods (e.g. borderBefore(), borderAfter()).
+    // They allow dirty layout even after DocumentLifecycle::LayoutClean which seems not proper. crbug.com/538236.
 
     mutable Vector<int> m_columnPos;
 
     // This Vector holds the columns counts over the entire table.
     //
     // To save memory at the expense of massive code complexity, the code tries
     // to coalesce columns. This means that we try to the wider column grouping
     // seen over the LayoutTableSections.
     //
     // Note that this is also a defensive pattern as <td colspan="6666666666">
@@ skipping 32 matching lines @@
     //
     // The layout algorithm is delegated to TableLayoutAlgorithm. This enables
     // changing 'table-layout' without having to reattach the <table>.
     //
     // As the algorithm is dependent on the style, this field is nullptr before
     // the first style is applied in styleDidChange().
     OwnPtr<TableLayoutAlgorithm> m_tableLayout;
 
     // A sorted list of all unique border values that we want to paint.
     CollapsedBorderValues m_collapsedBorders;
-    // Used by TablePainter during painting.
-    // TODO(wangxianzhu): Move it into painter.
-    mutable const CollapsedBorderValue* m_currentBorder;
     bool m_collapsedBordersValid : 1;
 
     mutable bool m_hasColElements : 1;
     mutable bool m_needsSectionRecalc : 1;
 
     bool m_columnLogicalWidthChanged : 1;
     mutable bool m_columnLayoutObjectsValid: 1;
     mutable bool m_hasCellColspanThatDeterminesTableWidth : 1;
     bool hasCellColspanThatDeterminesTableWidth() const
     {
@@ skipping 18 matching lines @@
     if (m_firstBody)
         return m_firstBody;
     return m_foot;
 }
 
 DEFINE_LAYOUT_OBJECT_TYPE_CASTS(LayoutTable, isTable());
 
 } // namespace blink
 
 #endif // LayoutTable_h