From 0bcd36838d6364ffe30a1e3aa9ce25bccfbd97f1 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Emilio=20Cobos=20=C3=81lvarez?= Date: Thu, 30 Nov 2017 17:26:12 +0100 Subject: [PATCH] style: Remove redundant style sharing check. We check that a few lines before calling test_candidate. --- components/style/sharing/mod.rs | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/components/style/sharing/mod.rs b/components/style/sharing/mod.rs index 360d644f407..79ddd58dfce 100644 --- a/components/style/sharing/mod.rs +++ b/components/style/sharing/mod.rs @@ -648,6 +648,8 @@ impl StyleSharingCache { nth_index_cache: &mut NthIndexCache, selector_flags_map: &mut SelectorFlagsMap ) -> Option { + debug_assert!(!target.is_native_anonymous()); + // Check that we have the same parent, or at least that the parents // share styles and permit sharing across their children. The latter // check allows us to share style between cousins if the parents @@ -657,13 +659,6 @@ impl StyleSharingCache { return None; } - if target.is_native_anonymous() { - debug_assert!(!candidate.element.is_native_anonymous(), - "Why inserting NAC into the cache?"); - trace!("Miss: Native Anonymous Content"); - return None; - } - // Note that in the XBL case, we should be able to assert that the // scopes are different, since two elements with different XBL bindings // need to necessarily have different style (and thus children of them