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

Unified Diff: third_party/WebKit/Source/core/css/resolver/StyleAdjuster.cpp

Issue 2286543002: Add Length::isPercent and use it in tables. (Closed)
Patch Set: rebase Created 4 years, 3 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/css/resolver/StyleAdjuster.cpp
diff --git a/third_party/WebKit/Source/core/css/resolver/StyleAdjuster.cpp b/third_party/WebKit/Source/core/css/resolver/StyleAdjuster.cpp
index 83ada0d1c1ff1e60c94ac40b967c080ed383f6d2..35509148ac0ab89ba718a446e33c0f733a6b6b26 100644
--- a/third_party/WebKit/Source/core/css/resolver/StyleAdjuster.cpp
+++ b/third_party/WebKit/Source/core/css/resolver/StyleAdjuster.cpp
@@ -322,9 +322,9 @@ static void adjustStyleForDisplay(ComputedStyle& style, const ComputedStyle& par
// We want to count vertical percentage paddings/margins on flex items because our current
// behavior is different from the spec and we want to gather compatibility data.
- if (style.paddingBefore().hasPercent() || style.paddingAfter().hasPercent())
+ if (style.paddingBefore().isPercentOrCalc() || style.paddingAfter().isPercentOrCalc())
UseCounter::count(document, UseCounter::FlexboxPercentagePaddingVertical);
- if (style.marginBefore().hasPercent() || style.marginAfter().hasPercent())
+ if (style.marginBefore().isPercentOrCalc() || style.marginAfter().isPercentOrCalc())
UseCounter::count(document, UseCounter::FlexboxPercentageMarginVertical);
}
}

Powered by Google App Engine
This is Rietveld 408576698