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

Unified Diff: sky/engine/core/dom/Element.cpp

Issue 858423002: Don't store a RuleFeatureSet on ScopedStyleResolver. (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: better test names. Created 5 years, 11 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « sky/engine/core/css/resolver/ScopedStyleResolver.cpp ('k') | sky/engine/core/dom/StyleEngine.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sky/engine/core/dom/Element.cpp
diff --git a/sky/engine/core/dom/Element.cpp b/sky/engine/core/dom/Element.cpp
index f5257ef68af0358adc7d4a0eaca757b9a7ad6ee2..c82a7d9192882732d05af4943a0aedf5ecd82eff 100644
--- a/sky/engine/core/dom/Element.cpp
+++ b/sky/engine/core/dom/Element.cpp
@@ -1760,11 +1760,11 @@ bool Element::affectedByAttributeSelector(const AtomicString& attributeName) con
{
if (attributeName.isEmpty())
return false;
- if (treeScope().scopedStyleResolver().features().hasSelectorForAttribute(attributeName))
+ if (treeScope().scopedStyleResolver().hasSelectorForAttribute(attributeName))
return true;
// Host rules could also have effects.
if (ShadowRoot* shadowRoot = this->shadowRoot())
- return shadowRoot->scopedStyleResolver().features().hasSelectorForAttribute(attributeName);
+ return shadowRoot->scopedStyleResolver().hasSelectorForAttribute(attributeName);
return false;
}
@@ -1772,11 +1772,11 @@ bool Element::affectedByClassSelector(const AtomicString& classValue) const
{
if (classValue.isEmpty())
return false;
- if (treeScope().scopedStyleResolver().features().hasSelectorForClass(classValue))
+ if (treeScope().scopedStyleResolver().hasSelectorForClass(classValue))
return true;
// Host rules could also have effects.
if (ShadowRoot* shadowRoot = this->shadowRoot())
- return shadowRoot->scopedStyleResolver().features().hasSelectorForClass(classValue);
+ return shadowRoot->scopedStyleResolver().hasSelectorForClass(classValue);
return false;
}
@@ -1784,11 +1784,11 @@ bool Element::affectedByIdSelector(const AtomicString& idValue) const
{
if (idValue.isEmpty())
return false;
- if (treeScope().scopedStyleResolver().features().hasSelectorForId(idValue))
+ if (treeScope().scopedStyleResolver().hasSelectorForId(idValue))
return true;
// Host rules could also have effects.
if (ShadowRoot* shadowRoot = this->shadowRoot())
- return shadowRoot->scopedStyleResolver().features().hasSelectorForId(idValue);
+ return shadowRoot->scopedStyleResolver().hasSelectorForId(idValue);
return false;
}
« no previous file with comments | « sky/engine/core/css/resolver/ScopedStyleResolver.cpp ('k') | sky/engine/core/dom/StyleEngine.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698