[css-align] Implement place-content alignment shorthand

third_party/WebKit/Source/core/css/ComputedStyleCSSValueMapping.cpp

Index: third_party/WebKit/Source/core/css/ComputedStyleCSSValueMapping.cpp
diff --git a/third_party/WebKit/Source/core/css/ComputedStyleCSSValueMapping.cpp b/third_party/WebKit/Source/core/css/ComputedStyleCSSValueMapping.cpp
index e50a948574f2ca8cd96a776b7bac68d06d7fc228..0de9801fbec42fa06b71349f020d52e3d3293cb8 100644
--- a/third_party/WebKit/Source/core/css/ComputedStyleCSSValueMapping.cpp
+++ b/third_party/WebKit/Source/core/css/ComputedStyleCSSValueMapping.cpp
@@ -2286,6 +2286,13 @@ const CSSValue* ComputedStyleCSSValueMapping::get(
       return CSSIdentifierValue::create(style.display());
     case CSSPropertyEmptyCells:
       return CSSIdentifierValue::create(style.emptyCells());
+    case CSSPropertyPlaceContent: {
+      // TODO (jfernandez): The spec states that we should return the specific
+      // value.

[svillar, 2017/02/02 09:11:19]

You mean "specified" value?

Also in the tests you check the value of the longhands but not the computed
value of the shorthand.

[jfernandez, 2017/02/06 15:17:17]

Yes.
Actually I do; look at the calls to checkValues in the different Test functions.

+      return valuesForShorthandProperty(placeContentShorthand(), style,
+                                        layoutObject, styledNode,
+                                        allowVisitedStyle);
+    }
     case CSSPropertyAlignContent:
       return valueForContentPositionAndDistributionWithOverflowAlignment(
           style.alignContent(), CSSValueStretch);