Consider all the pseudo-classes in a given compound selector when computing restyle hints, not just the rightmost one.

For some reason when I wrote this code I mixed up the rules for pseudo-elements
(rightmost, maximum of one) with those of pseudo-classes (which have no such
restriction). This caused a correctness issue which can be demonstrated with the
associated reftest modification.

1 files changed, 10 insertions, 9 deletions

diff --git a/components/style/restyle_hints.rs b/components/style/restyle_hints.rs
index acb521b0f10..fe143c3e315 100644
--- a/components/style/restyle_hints.rs
+++ b/components/style/restyle_hints.rs
@@ -220,15 +220,16 @@ impl StateDependencySet {
         let mut cur = selector;
         let mut combinator: Option<Combinator> = None;
         loop {
-            if let Some(rightmost) = cur.simple_selectors.last() {
-                let state_dep = selector_to_state(rightmost);
-                if !state_dep.is_empty() {
-                    self.deps.push(StateDependency {
-                        selector: cur.clone(),
-                        combinator: combinator,
-                        state: state_dep,
-                    });
-                }
+            let mut deps = ElementState::empty();
+            for s in &cur.simple_selectors {
+                deps.insert(selector_to_state(s));
+            }
+            if !deps.is_empty() {
+                self.deps.push(StateDependency {
+                    selector: cur.clone(),
+                    combinator: combinator,
+                    state: deps,
+                });
             }
 
             cur = match cur.next {