diff options
| author | Martin Robinson <mrobinson@igalia.com> | 2025-03-21 15:55:00 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2025-03-21 14:55:00 +0000 |
| commit | 54244797688fbeec85baac6fcfc9b71d555bf72c (patch) | |
| tree | 5e9914cce8a83a8d8b171da0ead4d6f1cf725ca5 /components/compositing/tracing.rs | |
| parent | 1f232eb17c031fe18035a17249de99b2af6fa53e (diff) | |
| download | servo-54244797688fbeec85baac6fcfc9b71d555bf72c.tar.gz servo-54244797688fbeec85baac6fcfc9b71d555bf72c.zip | |
metrics: Simplify `ProgressiveWebMetrics` (#35985)
Simply how `ProgressiveWebMetrics` works:
1. Keep only a single struct instead of one in layout and one script
that both implement the `ProgressiveWebMetrics` trait. Since layout
and script are the same thread these can now just be a single
`ProgressiveWebMetrics` struct stored in script.
2. Have the compositor be responsible for informing the Constellation
(which informs the ScripThread) about paint metrics. This makes
communication flow one way and removes one dependency between the
compositor and script (of two).
3. All units tests are moved into the `metrics` crate itself since there
is only one struct there now.
Signed-off-by: Martin Robinson <mrobinson@igalia.com>
Diffstat (limited to 'components/compositing/tracing.rs')
| -rw-r--r-- | components/compositing/tracing.rs | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/components/compositing/tracing.rs b/components/compositing/tracing.rs index 1ef0b41971c..907c931aebe 100644 --- a/components/compositing/tracing.rs +++ b/components/compositing/tracing.rs @@ -39,7 +39,6 @@ mod from_constellation { Self::SetThrottled(..) => target!("SetThrottled"), Self::NewWebRenderFrameReady(..) => target!("NewWebRenderFrameReady"), Self::PipelineExited(..) => target!("PipelineExited"), - Self::PendingPaintMetric(..) => target!("PendingPaintMetric"), Self::LoadComplete(..) => target!("LoadComplete"), Self::WebDriverMouseButtonEvent(..) => target!("WebDriverMouseButtonEvent"), Self::WebDriverMouseMoveEvent(..) => target!("WebDriverMouseMoveEvent"), |