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

Unified Diff: third_party/WebKit/LayoutTests/fast/alignment/parse-place-items.html

Issue 2915773002: Reland of [css-align] Don't resolve 'auto' values for computed style. (Closed)
Patch Set: Applied suggested changes Created 3 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/LayoutTests/fast/alignment/parse-place-items.html
diff --git a/third_party/WebKit/LayoutTests/fast/alignment/parse-place-items.html b/third_party/WebKit/LayoutTests/fast/alignment/parse-place-items.html
index 407da5cc19f7f58c5d8e437f41e17b36e2720674..712bd1e93bf1f2ccc4c50a9e6aaa0391151ec9f3 100644
--- a/third_party/WebKit/LayoutTests/fast/alignment/parse-place-items.html
+++ b/third_party/WebKit/LayoutTests/fast/alignment/parse-place-items.html
@@ -58,6 +58,9 @@
#placeItemsStartSafe {
place-items: start safe;
}
+#placeItemsBaselineSafe {
+ place-items: baseline safe;
+}
#placeItemsStartEndLeft {
place-items: start end left;
}
@@ -207,8 +210,8 @@ test(function() {
}, "Test setting 'start safe' as incorrect value through CSS.");
test(function() {
- checkValues(placeItemsStartSafe, "placeItems", "place-items", "", "normal normal");
- checkPlaceItemsValues(placeItemsStartSafe, "", "normal", "normal");
+ checkValues(placeItemsBaselineSafe, "placeItems", "place-items", "", "normal normal");
+ checkPlaceItemsValues(placeItemsBaselineSafe, "", "normal", "normal");
}, "Test setting 'baseline safe' as incorrect value through CSS.");
test(function() {
@@ -224,16 +227,16 @@ test(function() {
}, "Test setting values through JS.");
test(function() {
- checkPlaceItemsValuesBadJS("auto normal", "normal", "normal");
- checkPlaceItemsValuesBadJS("space-between", "normal", "normal");
- checkPlaceItemsValuesBadJS("center safe", "normal", "normal");
- checkPlaceItemsValuesBadJS("center self-start center", "normal", "normal");
- checkPlaceItemsValuesBadJS("asrt", "normal", "normal");
- checkPlaceItemsValuesBadJS("auto", "normal", "normal");
- checkPlaceItemsValuesBadJS("10px", "normal", "normal");
- checkPlaceItemsValuesBadJS("stretch safe", "normal", "normal");
- checkPlaceItemsValuesBadJS("self-start start end", "normal", "normal");
- checkPlaceItemsValuesBadJS("", "normal", "normal");
+ checkPlaceItemsValuesBadJS("auto normal");
+ checkPlaceItemsValuesBadJS("space-between");
+ checkPlaceItemsValuesBadJS("center safe");
+ checkPlaceItemsValuesBadJS("center self-start center");
+ checkPlaceItemsValuesBadJS("asrt", "normal");
+ checkPlaceItemsValuesBadJS("auto", "normal");
+ checkPlaceItemsValuesBadJS("10px", "normal");
+ checkPlaceItemsValuesBadJS("stretch safe");
+ checkPlaceItemsValuesBadJS("self-start start end");
+ checkPlaceItemsValuesBadJS("");
}, "Test setting incorrect values through JS.");
test(function() {

Powered by Google App Engine
This is Rietveld 408576698