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

Source/core/css/CSSContentDistributionValue.h

+// Copyright (c) 2014 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE fil

[Julien - ping for review, 2014/10/28 17:23:05]

LICENSE fil? :)

[jfernandez, 2014/10/29 11:03:04]

Done.

+
+#ifndef CSSContentDistributionValue_h
+#define CSSContentDistributionValue_h
+
+#include "core/css/CSSValue.h"
+#include "core/css/CSSValuePool.h"
+#include "wtf/RefPtr.h"
+
+namespace blink {
+
+class CSSContentDistributionValue : public CSSValue {
+public:
+    static PassRefPtrWillBeRawPtr<CSSContentDistributionValue> create(CSSValueID distribution, CSSValueID position, CSSValueID overflow)
+    {
+        return adoptRefWillBeNoop(new CSSContentDistributionValue(distribution, position, overflow));
+    }
+    ~CSSContentDistributionValue();
+
+    PassRefPtrWillBeRawPtr<CSSPrimitiveValue> distribution() const { return cssValuePool().createIdentifierValue(m_distribution); }
+
+    PassRefPtrWillBeRawPtr<CSSPrimitiveValue> position() const { return cssValuePool().createIdentifierValue(m_position); }
+
+    PassRefPtrWillBeRawPtr<CSSPrimitiveValue> overflow() const { return cssValuePool().createIdentifierValue(m_overflow); }
+
+    String customCSSText() const;
+
+    bool equals(const CSSContentDistributionValue&) const;
+
+    void traceAfterDispatch(Visitor* visitor) { CSSValue::traceAfterDispatch(visitor); }

[Julien - ping for review, 2014/10/28 17:23:05]

Do we need that as it calls the base class?

[jfernandez, 2014/10/29 11:03:04]

Well, the class doesn't hold any reference to be handled by the GC, so I think
just calling the empty definition in the base class makes sense. 

The CSSContentDistributionValue implementation will be called by the generic
CSSValue::trace() method, so implementing the prototype for this class make
sense, as far as I could understand how olipan works. Adding Mads Ager to the CC
just in case he could provide any info on this.

+
+private:
+    explicit CSSContentDistributionValue(CSSValueID distribution, CSSValueID position, CSSValueID overflow);
+
+    CSSValueID m_distribution;
+    CSSValueID m_position;
+    CSSValueID m_overflow;
+};
+
+DEFINE_CSS_VALUE_TYPE_CASTS(CSSContentDistributionValue, isContentDistributionValue());
+
+} // namespace blink
+
+#endif // CSSContentDistributionValue_h