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

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

Issue 1583433002: [css-align] New CSS Value 'normal' for Content Alignment (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed layout tests failing. Created 4 years, 10 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 80bb11ba52461ac8e913c0406ca630cc7d76c7c4..4be8ff8fa37c716d5b1bfc6674c65fb58416cfba 100644
--- a/third_party/WebKit/Source/core/css/resolver/StyleAdjuster.cpp
+++ b/third_party/WebKit/Source/core/css/resolver/StyleAdjuster.cpp
@@ -289,32 +289,6 @@ void StyleAdjuster::adjustStyleForAlignment(ComputedStyle& style, const Computed
if (isFlexOrGrid)
style.setAlignItemsPosition(ItemPositionStretch);
}
-
- // Block Containers: For table cells, the behavior of the 'auto' depends on the computed
- // value of 'vertical-align', otherwise behaves as 'start'.
- // Flex Containers: 'auto' computes to 'flex-start'.
- // Grid Containers: 'auto' computes to 'start', and 'stretch' behaves like 'start'.
- if ((style.justifyContentPosition() == ContentPositionAuto) && (style.justifyContentDistribution() == ContentDistributionDefault)) {
- if (style.isDisplayFlexibleOrGridBox()) {
- if (style.isDisplayFlexibleBox())
- style.setJustifyContentPosition(ContentPositionFlexStart);
- else
- style.setJustifyContentPosition(ContentPositionStart);
- }
- }
-
- // Block Containers: For table cells, the behavior of the 'auto' depends on the computed
- // value of 'vertical-align', otherwise behaves as 'start'.
- // Flex Containers: 'auto' computes to 'stretch'.
- // Grid Containers: 'auto' computes to 'start', and 'stretch' behaves like 'start'.
- if (style.alignContentPosition() == ContentPositionAuto && style.alignContentDistribution() == ContentDistributionDefault) {
- if (style.isDisplayFlexibleOrGridBox()) {
- if (style.isDisplayFlexibleBox())
- style.setAlignContentDistribution(ContentDistributionStretch);
- else
- style.setAlignContentPosition(ContentPositionStart);
- }
- }
}
void StyleAdjuster::adjustStyleForHTMLElement(ComputedStyle& style, const ComputedStyle& parentStyle, HTMLElement& element)

Powered by Google App Engine
This is Rietveld 408576698