From 8c737a6a79fe591567c0fc265e21bc3631fb83e7 Mon Sep 17 00:00:00 2001 From: Oriol Brufau Date: Wed, 30 Jul 2025 14:55:25 +0200 Subject: [PATCH] layout: Make a couple tracing reflect what they actually trace (#38349) `IndependentFormattingContext::layout` was being traced with name `IndependentFormattingContext::layout_with_caching`. Better use `IndependentFormattingContext::layout` instead. There was also a debug trace with name `NonReplaced cache miss`, but now this code also applies to replaced boxes, so I'm renaming it to `IndependentFormattingContext::layout cache miss`. Testing: Not needed (no behavior change) Signed-off-by: Oriol Brufau --- components/layout/formatting_contexts.rs | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/components/layout/formatting_contexts.rs b/components/layout/formatting_contexts.rs index 577e0afbd50..16cef35bbbe 100644 --- a/components/layout/formatting_contexts.rs +++ b/components/layout/formatting_contexts.rs @@ -290,10 +290,7 @@ impl IndependentFormattingContext { } } - #[servo_tracing::instrument( - name = "IndependentFormattingContext::layout_with_caching", - skip_all - )] + #[servo_tracing::instrument(name = "IndependentFormattingContext::layout", skip_all)] #[allow(clippy::too_many_arguments)] pub(crate) fn layout( &self, @@ -317,7 +314,7 @@ impl IndependentFormattingContext { } #[cfg(feature = "tracing")] tracing::debug!( - name: "NonReplaced cache miss", + name: "IndependentFormattingContext::layout cache miss", cached = ?cache.containing_block_for_children_size, required = ?containing_block_for_children.size, );