CSS Custom Properties (Variables)

Source/core/css/CSSVariableData.h

+// Copyright 2015 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 file.
+
+#ifndef CSSVariableData_h
+#define CSSVariableData_h
+
+#include "core/css/parser/CSSTokenizer.h"
+#include "wtf/RefCounted.h"
+#include "wtf/text/WTFString.h"
+
+namespace blink {
+
+class CSSParserTokenRange;
+
+class CSSVariableData : public RefCounted<CSSVariableData> {
+    WTF_MAKE_NONCOPYABLE(CSSVariableData);
+    WTF_MAKE_FAST_ALLOCATED(CSSVariableData);
+public:
+    static PassRefPtr<CSSVariableData> create(const CSSParserTokenRange& range, bool needsVariableResolution = true)
+    {
+        return adoptRef(new CSSVariableData(range, needsVariableResolution));
+    }
+
+    Vector<CSSParserToken>& tokens() { return m_tokens; }
+
+    bool needsVariableResolution() const { return m_needsVariableResolution; }
+    void setNeedsVariableResolution(bool needsVariableResolution) { m_needsVariableResolution = needsVariableResolution; }
+
+    String string() const { return m_string; }
+private:
+    CSSVariableData(const CSSParserTokenRange&, bool needsVariableResolution);
+
+    template<typename CharacterType> void consumeAndUpdateTokens(const CSSParserTokenRange&);
+
+    String m_string;
+    Vector<String> m_stringPool;
+    Vector<CSSParserToken> m_tokens;
+    bool m_needsVariableResolution : 1;
+    bool m_important : 1;

[alancutter (OOO until 2018), 2015/07/14 06:12:57]

Is m_important ever used? Shouldn't this bool be passed to CSSProperty() in
CSSParserImpl::consumeVariableDeclarationValue() instead of stored here? I
assume our property application ordering is set up to use that.

+};
+
+} // namespace blink
+
+#endif // CSSVariableData_h