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

Side by Side Diff: sky/engine/core/html/HTMLContentElement.cpp

Issue 801283006: Get rid of SelectorCheckingContext. (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: missing static. 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 unified diff | Download patch
« no previous file with comments | « sky/engine/core/dom/SelectorQuery.cpp ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2011 Google Inc. All rights reserved. 2 * Copyright (C) 2011 Google Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions are 5 * modification, are permitted provided that the following conditions are
6 * met: 6 * met:
7 * 7 *
8 * * Redistributions of source code must retain the above copyright 8 * * Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * * Neither the name of Google Inc. nor the names of its 10 * * Neither the name of Google Inc. nor the names of its
(...skipping 89 matching lines...) Expand 10 before | Expand all | Expand 10 after
100 if (allowAnyPseudoClasses) 100 if (allowAnyPseudoClasses)
101 continue; 101 continue;
102 for (const CSSSelector* subSelector = selector; subSelector; subSelector = subSelector->tagHistory()) { 102 for (const CSSSelector* subSelector = selector; subSelector; subSelector = subSelector->tagHistory()) {
103 if (includesDisallowedPseudoClass(*subSelector)) 103 if (includesDisallowedPseudoClass(*subSelector))
104 return false; 104 return false;
105 } 105 }
106 } 106 }
107 return true; 107 return true;
108 } 108 }
109 109
110 static inline bool checkOneSelector(const CSSSelector& selector, const Vector<Ra wPtr<Node>, 32>& siblings, int nth)
111 {
112 Element* element = toElement(siblings[nth]);
113 SelectorChecker selectorChecker;
114 SelectorChecker::SelectorCheckingContext context(selector, element);
115 return selectorChecker.match(context);
116 }
117
118 bool HTMLContentElement::matchSelector(const Vector<RawPtr<Node>, 32>& siblings, int nth) const 110 bool HTMLContentElement::matchSelector(const Vector<RawPtr<Node>, 32>& siblings, int nth) const
119 { 111 {
120 for (const CSSSelector* selector = selectorList().first(); selector; selecto r = CSSSelectorList::next(*selector)) { 112 for (const CSSSelector* selector = selectorList().first(); selector; selecto r = CSSSelectorList::next(*selector)) {
121 if (checkOneSelector(*selector, siblings, nth)) 113 Element* element = toElement(siblings[nth]);
114 SelectorChecker checker(*element);
115 if (checker.match(*selector, element))
122 return true; 116 return true;
123 } 117 }
124 return false; 118 return false;
125 } 119 }
126 120
127 } 121 }
OLDNEW
« no previous file with comments | « sky/engine/core/dom/SelectorQuery.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698