From 5ecf1508eefc580e13b700eef4b27527e158f46b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Emilio=20Cobos=20=C3=81lvarez?= Date: Tue, 7 Nov 2017 15:58:02 +0100 Subject: [PATCH] style: We never call accumulate_damage_for in a forgetful traversal. We early-return instead before calling it. MozReview-Commit-ID: JaRWPHCQAl9 --- components/style/matching.rs | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/components/style/matching.rs b/components/style/matching.rs index 5f140b54ca0..848fd77f9b2 100644 --- a/components/style/matching.rs +++ b/components/style/matching.rs @@ -340,12 +340,7 @@ trait PrivateMatchMethods: TElement { pseudo: Option<&PseudoElement>, ) -> ChildCascadeRequirement { debug!("accumulate_damage_for: {:?}", self); - - // Don't accumulate damage if we're in a forgetful traversal. - if shared_context.traversal_flags.contains(TraversalFlags::Forgetful) { - debug!(" > forgetful traversal"); - return ChildCascadeRequirement::MustCascadeChildren; - } + debug_assert!(!shared_context.traversal_flags.contains(TraversalFlags::Forgetful)); let difference = self.compute_style_difference(old_values, new_values, pseudo);