[CSS Grid Layout] Upgrade justify-content parsing to CSS3 Box Alignment spec.

Source/core/css/resolver/StyleBuilderCustom.cpp

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 2004-2005 Allan Sandfeld Jensen (kde@carewolf.com)
  * Copyright (C) 2006, 2007 Nicholas Shanks (webkit@nickshanks.com)
  * Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010, 2011, 2012, 2013 Apple Inc. All rights reserved.
  * Copyright (C) 2007 Alexey Proskuryakov <ap@webkit.org>
  * Copyright (C) 2007, 2008 Eric Seidel <eric@webkit.org>
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  * Copyright (c) 2011, Code Aurora Forum. All rights reserved.
@@ skipping 171 matching lines @@
             state.style()->setJustifyItems(*pairValue->second());
         } else {
             state.style()->setJustifyItems(*pairValue->first());
             state.style()->setJustifyItemsOverflowAlignment(*pairValue->second());
         }
     } else {
         state.style()->setJustifyItems(*primitiveValue);
     }
 }
 
+void StyleBuilderFunctions::applyInitialCSSPropertyJustifyContent(StyleResolverState& state)
+{
+    state.style()->setJustifyContent(RenderStyle::initialJustifyContent());
+    state.style()->setJustifyContentOverflowAlignment(RenderStyle::initialJustifyContentOverflowAlignment());
+    state.style()->setJustifyContentDistribution(RenderStyle::initialJustifyContentDistribution());
+}
+
+void StyleBuilderFunctions::applyInheritCSSPropertyJustifyContent(StyleResolverState& state)
+{
+    state.style()->setJustifyContent(state.parentStyle()->justifyContent());
+    state.style()->setJustifyContentOverflowAlignment(state.parentStyle()->justifyContentOverflowAlignment());
+    state.style()->setJustifyContentDistribution(state.parentStyle()->justifyContentDistribution());
+}
+
+void StyleBuilderFunctions::applyValueCSSPropertyJustifyContent(StyleResolverState& state, CSSValue* value)
+{
+    CSSPrimitiveValue* primitiveValue = toCSSPrimitiveValue(value);
+    if (Pair* pairValue = primitiveValue->getPairValue()) {
+        if (Pair* nestedPairValue = pairValue->first()->getPairValue()) {
+            state.style()->setJustifyContentDistribution(*nestedPairValue->first());
+            state.style()->setJustifyContent(*nestedPairValue->second());
+            state.style()->setJustifyContentOverflowAlignment(*pairValue->second());
+        } else if (CSSPrimitiveValue::isContentDistributionKeyword(pairValue->first()->getValueID())) {
+            state.style()->setJustifyContentDistribution(*pairValue->first());
+            if (CSSPrimitiveValue::isContentPositionKeyword(pairValue->second()->getValueID()))
+                state.style()->setJustifyContent(*pairValue->second());
+            else
+                state.style()->setJustifyContentOverflowAlignment(*pairValue->second());
+        } else {

[Timothy Loh, 2014/10/21 03:51:09]

Seems weird to not set all three values in these cases.

[jfernandez, 2014/10/27 11:44:37]

Well, only explicit values are set. The ones not set remain with the initial
values defined in RenderStyle.

[Timothy Loh, 2014/10/27 11:55:30]

We can go through this function multiple times for a single RenderStyle since we
don't have a cascade pass yet (we'll currently just call this for all matching
declarations in increasing priority order), so they won't necessarily be the
initial values.

[jfernandez, 2014/10/27 14:09:24]

Oh, I see now. I'll assign the 2 values as you suggested in the next patch.
Thanks for the comments.

+            state.style()->setJustifyContent(*pairValue->first());
+            state.style()->setJustifyContentOverflowAlignment(*pairValue->second());
+        }
+    } else {
+        if (CSSPrimitiveValue::isContentDistributionKeyword(primitiveValue->getValueID())) {

[Julien - ping for review, 2014/10/20 21:24:02]

I usually say that we control the format that gets passed here, which is why we
should be able to get away without adding the CSSPrimitiveValue functions.

A good way to do that would be to define a custom CSSValue that would wrap the
parsed data.

[jfernandez, 2014/10/27 11:44:37]

Good idea, thanks. Ill try that approach in the next patch.

+            state.style()->setJustifyContentDistribution(*primitiveValue);
+        } else {
+            state.style()->setJustifyContent(*primitiveValue);
+        }
+    }
+}
+
 void StyleBuilderFunctions::applyInitialCSSPropertyCursor(StyleResolverState& state)
 {
     state.style()->clearCursorList();
     state.style()->setCursor(RenderStyle::initialCursor());
 }
 
 void StyleBuilderFunctions::applyInheritCSSPropertyCursor(StyleResolverState& state)
 {
     state.style()->setCursor(state.parentStyle()->cursor());
     state.style()->setCursorList(state.parentStyle()->cursors());
@@ skipping 997 matching lines @@
         break;
     default:
         ASSERT_NOT_REACHED();
         break;
     }
 
     state.style()->setGridAutoFlow(autoFlow);
 }
 
 } // namespace blink