From c3501084f0e5eb62e816779b6e22e429da9a7fbd Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Emilio=20Cobos=20=C3=81lvarez?= Date: Mon, 7 Aug 2017 16:23:01 +0200 Subject: [PATCH] style: Don't pre-fill eager pseudo-maps. This was needed long time ago for Servo's layout. But I don't think there's any reason for doing this now. --- components/style/stylist.rs | 14 +++----------- 1 file changed, 3 insertions(+), 11 deletions(-) diff --git a/components/style/stylist.rs b/components/style/stylist.rs index f0b2dcc462c..9426ecd11f7 100644 --- a/components/style/stylist.rs +++ b/components/style/stylist.rs @@ -1646,14 +1646,10 @@ struct PerOriginCascadeData { impl PerOriginCascadeData { fn new() -> Self { - let mut data = PerOriginCascadeData { + Self { element_map: SelectorMap::new(), pseudos_map: Default::default(), - }; - SelectorImpl::each_eagerly_cascaded_pseudo_element(|pseudo| { - data.pseudos_map.insert(pseudo, SelectorMap::new()); - }); - data + } } #[inline] @@ -1677,11 +1673,7 @@ impl PerOriginCascadeData { } fn clear(&mut self) { - self.element_map = SelectorMap::new(); - self.pseudos_map = Default::default(); - SelectorImpl::each_eagerly_cascaded_pseudo_element(|pseudo| { - self.pseudos_map.insert(pseudo, SelectorMap::new()); - }); + *self = Self::new(); } fn has_rules_for_pseudo(&self, pseudo: &PseudoElement) -> bool {