| commit | c40b51a7fcf20ce04022a0f5cbfc957ee9e1aaed | [log] [tgz] |
|---|---|---|
| author | Simon Zünd <szuend@chromium.org> | Tue Sep 09 11:04:21 2025 |
| committer | Devtools-frontend LUCI CQ <devtools-frontend-scoped@luci-project-accounts.iam.gserviceaccount.com> | Tue Sep 09 12:55:13 2025 |
| tree | 761d964cb3579eb0fbbe1b720dbeb63839de6ce8 | |
| parent | 815772d12582dde1bee2c92caecfab9261aa63c6 [diff] |
[recorder] Don't emit UMA for empty replay speed As seen by Chromium error logs when running e2e tests, we attempt to emit UMA for 'undefined' replay speeds. R=alexrudenko@chromium.org Bug: None Change-Id: Ibc9c30e4674d40519ec2ee0414defce90a2e3c6c Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6929943 Commit-Queue: Alex Rudenko <alexrudenko@chromium.org> Commit-Queue: Simon Zünd <szuend@chromium.org> Reviewed-by: Alex Rudenko <alexrudenko@chromium.org> Auto-Submit: Simon Zünd <szuend@chromium.org>
diff --git a/front_end/panels/recorder/components/ReplaySection.ts b/front_end/panels/recorder/components/ReplaySection.ts index 181fc78..faa28a5 100644 --- a/front_end/panels/recorder/components/ReplaySection.ts +++ b/front_end/panels/recorder/components/ReplaySection.ts
@@ -168,7 +168,9 @@ this.#settings.replayExtension = ''; } - Host.userMetrics.recordingReplaySpeed(replaySpeedToMetricSpeedMap[speed]); + if (replaySpeedToMetricSpeedMap[speed]) { + Host.userMetrics.recordingReplaySpeed(replaySpeedToMetricSpeedMap[speed]); + } void ComponentHelpers.ScheduledRender.scheduleRender( this, this.#render,