diff options
author | Emilio Cobos Álvarez <emilio@crisal.io> | 2017-05-15 23:52:09 +0200 |
---|---|---|
committer | Emilio Cobos Álvarez <emilio@crisal.io> | 2017-05-17 10:58:24 +0200 |
commit | 522f8489d6fca474e6aad1d341ee84a99c130738 (patch) | |
tree | deac46958e2f132881752f42a958b15f679a84d1 /components/script/dom/node.rs | |
parent | 837531992864f920342020462830b933d5ed0280 (diff) | |
download | servo-522f8489d6fca474e6aad1d341ee84a99c130738.tar.gz servo-522f8489d6fca474e6aad1d341ee84a99c130738.zip |
Bug 1364850: Move PseudoElement to be just another combinator in selectors. r=bholley
MozReview-Commit-ID: 8OoOIodkKJ5
Signed-off-by: Emilio Cobos Álvarez <emilio@crisal.io>
Diffstat (limited to 'components/script/dom/node.rs')
-rw-r--r-- | components/script/dom/node.rs | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/components/script/dom/node.rs b/components/script/dom/node.rs index bec7a34d63f..5ebedf30bcd 100644 --- a/components/script/dom/node.rs +++ b/components/script/dom/node.rs @@ -68,7 +68,7 @@ use script_layout_interface::{LayoutElementType, LayoutNodeType, TrustedNodeAddr use script_layout_interface::message::Msg; use script_traits::DocumentActivity; use script_traits::UntrustedNodeAddress; -use selectors::matching::matches_selector_list; +use selectors::matching::{matches_selector_list, MatchingContext, MatchingMode}; use selectors::parser::SelectorList; use servo_url::ServoUrl; use std::borrow::ToOwned; @@ -341,11 +341,14 @@ impl<'a> Iterator for QuerySelectorIterator { fn next(&mut self) -> Option<Root<Node>> { let selectors = &self.selectors.0; + // TODO(cgaebel): Is it worth it to build a bloom filter here // (instead of passing `None`)? Probably. + let mut ctx = MatchingContext::new(MatchingMode::Normal, None); + self.iterator.by_ref().filter_map(|node| { if let Some(element) = Root::downcast(node) { - if matches_selector_list(selectors, &element, None) { + if matches_selector_list(selectors, &element, &mut ctx) { return Some(Root::upcast(element)); } } @@ -707,8 +710,9 @@ impl Node { Err(()) => Err(Error::Syntax), // Step 3. Ok(selectors) => { + let mut ctx = MatchingContext::new(MatchingMode::Normal, None); Ok(self.traverse_preorder().filter_map(Root::downcast).find(|element| { - matches_selector_list(&selectors.0, element, None) + matches_selector_list(&selectors.0, element, &mut ctx) })) } } |