Chromium Code Reviews| Index: third_party/WebKit/Source/core/layout/LayoutTable.h |
| diff --git a/third_party/WebKit/Source/core/layout/LayoutTable.h b/third_party/WebKit/Source/core/layout/LayoutTable.h |
| index 84ac080ddffdc1bca7166150633991862a94b6ac..f9aa28c377a72cd598222f18dee3f9e15621167a 100644 |
| --- a/third_party/WebKit/Source/core/layout/LayoutTable.h |
| +++ b/third_party/WebKit/Source/core/layout/LayoutTable.h |
| @@ -304,10 +304,6 @@ public: |
| 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 |
| @@ -384,7 +380,7 @@ private: |
| void recalcCollapsedBordersIfNeeded(); |
| - // TODO(layout-dev): All mutables in this class (except for m_currentBorder) are lazily updated by recalcSections() |
| + // TODO(layout-dev): All mutables in this class (except noted) are lazily updated by recalcSections() |
|
pdr.
2015/10/02 22:05:08
Hmm... what does "except noted" mean?
Can this ju
Xianzhu
2015/10/02 22:20:10
Done.
(Thought in case we add a mutable for anoth
|
| // 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. |
| @@ -439,9 +435,6 @@ private: |
| // 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; |