Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(234)

Unified Diff: third_party/WebKit/Source/core/layout/LayoutTable.h

Issue 2850633003: Rename LayoutTable::CollapseBorders() to ShouldCollapseBorders() (Closed)
Patch Set: Rebase Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/layout/LayoutTable.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 8d676e1211d615a0edea744d743cb4d6fad1f26a..e423b0caeacdc14084fa359ee163cf3e919589ce 100644
--- a/third_party/WebKit/Source/core/layout/LayoutTable.h
+++ b/third_party/WebKit/Source/core/layout/LayoutTable.h
@@ -145,7 +145,7 @@ class CORE_EXPORT LayoutTable final : public LayoutBlock {
int HBorderSpacing() const { return h_spacing_; }
int VBorderSpacing() const { return v_spacing_; }
- bool CollapseBorders() const {
+ bool ShouldCollapseBorders() const {
return Style()->BorderCollapse() == EBorderCollapse::kCollapse;
}
@@ -349,9 +349,9 @@ class CORE_EXPORT LayoutTable final : public LayoutBlock {
// 'border-spacing' only applies to separate borders (see 17.6.1 The
// separated borders model).
return BorderStart() + BorderEnd() +
- (CollapseBorders() ? LayoutUnit()
- : (PaddingStart() + PaddingEnd() +
- BorderSpacingInRowDirection()));
+ (ShouldCollapseBorders() ? LayoutUnit()
+ : (PaddingStart() + PaddingEnd() +
+ BorderSpacingInRowDirection()));
}
// Return the first column or column-group.
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/layout/LayoutTable.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698