Refactor StyleVariableData to fallback on root map for performance reasons.

third_party/WebKit/Source/core/style/StyleVariableData.cpp

 // 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.
 
 #include "core/style/StyleVariableData.h"
 
 #include "core/style/DataEquivalency.h"
 
 namespace blink {
 
 bool StyleVariableData::operator==(const StyleVariableData& other) const
 {
-    if (m_data.size() != other.m_data.size()) {
+    // It's technically possible for divergent roots to be value-equal,
+    // but unlikely. This equality operator is used for optimization purposes
+    // so it's OK to be occasionally wrong.
+    // TODO(shanestephens): Rename this to something that indicates it may not
+    // always return equality.
+    if (m_root != other.m_root)
         return false;
-    }
+
+    if (m_data.size() != other.m_data.size())
+        return false;
 
     for (const auto& iter : m_data) {
         RefPtr<CSSVariableData> otherData = other.m_data.get(iter.key);
         if (!dataEquivalent(iter.value, otherData))
             return false;
     }
 
     return true;
 }
 
+PassRefPtr<StyleVariableData> StyleVariableData::copy()
+{
+    return adoptRef(new StyleVariableData(*this));

[Timothy Loh, 2016/04/05 03:13:37]

No point moving this if it didn't change

[shans, 2016/04/08 00:55:49]

Done.

+}
+
+StyleVariableData::StyleVariableData(StyleVariableData& other)
+{
+    if (!other.m_root) {
+        m_root = &other;
+    } else {
+        m_data = other.m_data;
+        m_root = other.m_root;
+    }
+}
+

[Timothy Loh, 2016/04/05 03:13:37]

extra whitespace here

[shans, 2016/04/08 00:55:49]

Done.

+
+CSSVariableData* StyleVariableData::getVariable(const AtomicString& name) const
+{
+    auto result = m_data.find(name);
+    if (result == m_data.end() && m_root)
+        return m_root->getVariable(name);
+    if (result == m_data.end())
+        return nullptr;
+    return result->value.get();
+}
+
+

[Timothy Loh, 2016/04/05 03:13:37]

more extra whitespace here

[shans, 2016/04/08 00:55:49]

Done.

+
 } // namespace blink