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

Unified Diff: third_party/WebKit/Source/core/layout/LayoutMultiColumnSet.cpp

Issue 1956373002: Remove ColumnFill runtime setting. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 7 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
Index: third_party/WebKit/Source/core/layout/LayoutMultiColumnSet.cpp
diff --git a/third_party/WebKit/Source/core/layout/LayoutMultiColumnSet.cpp b/third_party/WebKit/Source/core/layout/LayoutMultiColumnSet.cpp
index bfad057c0861b8bc6ab10ceb0aa4b2d0d88d3a39..c2ffb57d6b0f1d4e116b43903d8f17fd1b3ab2ff 100644
--- a/third_party/WebKit/Source/core/layout/LayoutMultiColumnSet.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutMultiColumnSet.cpp
@@ -277,10 +277,8 @@ bool LayoutMultiColumnSet::heightIsAuto() const
// even if column-fill isn't 'balance' - in accordance with the spec). Pretending that
// column-fill is auto also matches the old multicol implementation, which has no support
// for this property.
- if (RuntimeEnabledFeatures::columnFillEnabled()) {
- if (multiColumnBlockFlow()->style()->getColumnFill() == ColumnFillBalance)
- return true;
- }
+ if (multiColumnBlockFlow()->style()->getColumnFill() == ColumnFillBalance)
+ return true;
if (LayoutBox* next = nextSiblingBox()) {
if (next->isLayoutMultiColumnSpannerPlaceholder()) {
// If we're followed by a spanner, we need to balance.

Powered by Google App Engine
This is Rietveld 408576698