Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1371)

Side by Side Diff: third_party/WebKit/Source/core/css/resolver/CSSVariableResolver.cpp

Issue 2676993002: Move PropertyRegistry::Registration into PropertyRegistration (Closed)
Patch Set: Rebased Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "core/css/resolver/CSSVariableResolver.h" 5 #include "core/css/resolver/CSSVariableResolver.h"
6 6
7 #include "core/CSSPropertyNames.h" 7 #include "core/CSSPropertyNames.h"
8 #include "core/CSSValueKeywords.h" 8 #include "core/CSSValueKeywords.h"
9 #include "core/StyleBuilderFunctions.h" 9 #include "core/StyleBuilderFunctions.h"
10 #include "core/StylePropertyShorthand.h" 10 #include "core/StylePropertyShorthand.h"
(...skipping 27 matching lines...) Expand all
38 } 38 }
39 39
40 CSSVariableData* CSSVariableResolver::valueForCustomProperty( 40 CSSVariableData* CSSVariableResolver::valueForCustomProperty(
41 AtomicString name) { 41 AtomicString name) {
42 if (m_variablesSeen.contains(name)) { 42 if (m_variablesSeen.contains(name)) {
43 m_cycleStartPoints.insert(name); 43 m_cycleStartPoints.insert(name);
44 return nullptr; 44 return nullptr;
45 } 45 }
46 46
47 DCHECK(m_registry || !RuntimeEnabledFeatures::cssVariables2Enabled()); 47 DCHECK(m_registry || !RuntimeEnabledFeatures::cssVariables2Enabled());
48 const PropertyRegistry::Registration* registration = 48 const PropertyRegistration* registration =
49 m_registry ? m_registry->registration(name) : nullptr; 49 m_registry ? m_registry->registration(name) : nullptr;
50 50
51 CSSVariableData* variableData = nullptr; 51 CSSVariableData* variableData = nullptr;
52 if (!registration || registration->inherits()) { 52 if (!registration || registration->inherits()) {
53 if (m_inheritedVariables) 53 if (m_inheritedVariables)
54 variableData = m_inheritedVariables->getVariable(name); 54 variableData = m_inheritedVariables->getVariable(name);
55 } else { 55 } else {
56 if (m_nonInheritedVariables) 56 if (m_nonInheritedVariables)
57 variableData = m_nonInheritedVariables->getVariable(name); 57 variableData = m_nonInheritedVariables->getVariable(name);
58 } 58 }
(...skipping 272 matching lines...) Expand 10 before | Expand all | Expand 10 after
331 CSSVariableResolver::CSSVariableResolver(const StyleResolverState& state) 331 CSSVariableResolver::CSSVariableResolver(const StyleResolverState& state)
332 : m_inheritedVariables(state.style()->inheritedVariables()), 332 : m_inheritedVariables(state.style()->inheritedVariables()),
333 m_nonInheritedVariables(state.style()->nonInheritedVariables()), 333 m_nonInheritedVariables(state.style()->nonInheritedVariables()),
334 m_registry(state.document().propertyRegistry()) {} 334 m_registry(state.document().propertyRegistry()) {}
335 335
336 DEFINE_TRACE(CSSVariableResolver) { 336 DEFINE_TRACE(CSSVariableResolver) {
337 visitor->trace(m_registry); 337 visitor->trace(m_registry);
338 } 338 }
339 339
340 } // namespace blink 340 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698