Revert "[WebAudio] Implement AudioContext.playbackStats behind a flag" [chromium/src : main]

38 views
Skip to first unread message

Sergey Poromov (Gerrit)

unread,
Feb 5, 2026, 2:49:19 AM (6 days ago) Feb 5
to Rubber Stamper, Chromium LUCI CQ, Hongchan Choi, Kent Tamura, AyeAye, Blink W3C Test Autoroller, Olga Sharonova, Raphael Kubo da Costa, AI Code Reviewer, Fredrik Hernqvist, chromium...@chromium.org, Kentaro Hara, Chromium Metrics Reviews, asvitkine...@chromium.org, blink-revie...@chromium.org, android-web...@chromium.org, ashleynewson+w...@chromium.org, jmedle...@chromium.org, kinuko...@chromium.org, blink-revie...@chromium.org, blink-re...@chromium.org, blink-...@chromium.org
Attention needed from Hongchan Choi, Kent Tamura and Michael Wilson

Sergey Poromov voted Auto-Submit+1

Auto-Submit+1
Open in Gerrit

Related details

Attention is currently required from:
  • Hongchan Choi
  • Kent Tamura
  • Michael Wilson
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement is not satisfiedCode-Owners
  • requirement is not satisfiedCode-Review
  • requirement is not satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: chromium/src
Gerrit-Branch: main
Gerrit-Change-Id: I735fc8eb5680e21f606b4866236434898bf20b02
Gerrit-Change-Number: 7543642
Gerrit-PatchSet: 1
Gerrit-Owner: Sergey Poromov <por...@chromium.org>
Gerrit-Reviewer: Chromium LUCI CQ <chromiu...@luci-project-accounts.iam.gserviceaccount.com>
Gerrit-Reviewer: Hongchan Choi <hong...@chromium.org>
Gerrit-Reviewer: Kent Tamura <tk...@chromium.org>
Gerrit-Reviewer: Michael Wilson <mjwi...@chromium.org>
Gerrit-Reviewer: Rubber Stamper <rubber-...@appspot.gserviceaccount.com>
Gerrit-Reviewer: Sergey Poromov <por...@chromium.org>
Gerrit-CC: AI Code Reviewer <peep-gen...@system.gserviceaccount.com>
Gerrit-CC: Blink W3C Test Autoroller <blink-w3c-te...@chromium.org>
Gerrit-CC: Chromium Metrics Reviews <chromium-met...@google.com>
Gerrit-CC: Fredrik Hernqvist <fhern...@google.com>
Gerrit-CC: Kentaro Hara <har...@chromium.org>
Gerrit-CC: Olga Sharonova <ol...@google.com>
Gerrit-CC: Raphael Kubo da Costa <ku...@igalia.com>
Gerrit-Attention: Michael Wilson <mjwi...@chromium.org>
Gerrit-Attention: Hongchan Choi <hong...@chromium.org>
Gerrit-Attention: Kent Tamura <tk...@chromium.org>
Gerrit-Comment-Date: Thu, 05 Feb 2026 07:49:07 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
unsatisfied_requirement
open
diffy

Rubber Stamper (Gerrit)

unread,
Feb 5, 2026, 2:50:15 AM (6 days ago) Feb 5
to Sergey Poromov, Kent Tamura, Chromium LUCI CQ, Hongchan Choi, AyeAye, Blink W3C Test Autoroller, Olga Sharonova, Raphael Kubo da Costa, AI Code Reviewer, Fredrik Hernqvist, chromium...@chromium.org, Kentaro Hara, Chromium Metrics Reviews, asvitkine...@chromium.org, blink-revie...@chromium.org, android-web...@chromium.org, ashleynewson+w...@chromium.org, jmedle...@chromium.org, kinuko...@chromium.org, blink-revie...@chromium.org, blink-re...@chromium.org, blink-...@chromium.org
Attention needed from Hongchan Choi, Michael Wilson and Sergey Poromov

Rubber Stamper voted

Bot-Commit+1
Commit-Queue+2
Open in Gerrit

Related details

Attention is currently required from:
  • Hongchan Choi
  • Michael Wilson
  • Sergey Poromov
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement satisfiedCode-Owners
  • requirement satisfiedCode-Review
  • requirement satisfiedReview-Enforcement
Gerrit-Attention: Sergey Poromov <por...@chromium.org>
Gerrit-Comment-Date: Thu, 05 Feb 2026 07:50:03 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy

Kent Tamura (Gerrit)

unread,
Feb 5, 2026, 2:50:18 AM (6 days ago) Feb 5
to Sergey Poromov, Kent Tamura, Rubber Stamper, Chromium LUCI CQ, Hongchan Choi, AyeAye, Blink W3C Test Autoroller, Olga Sharonova, Raphael Kubo da Costa, AI Code Reviewer, Fredrik Hernqvist, chromium...@chromium.org, Kentaro Hara, Chromium Metrics Reviews, asvitkine...@chromium.org, blink-revie...@chromium.org, android-web...@chromium.org, ashleynewson+w...@chromium.org, jmedle...@chromium.org, kinuko...@chromium.org, blink-revie...@chromium.org, blink-re...@chromium.org, blink-...@chromium.org
Attention needed from Hongchan Choi, Michael Wilson and Sergey Poromov

Kent Tamura voted

Code-Review+1
Commit-Queue+2
Gerrit-Comment-Date: Thu, 05 Feb 2026 07:49:49 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy

Chromium LUCI CQ (Gerrit)

unread,
Feb 5, 2026, 2:50:33 AM (6 days ago) Feb 5
to Sergey Poromov, Rubber Stamper, Kent Tamura, Hongchan Choi, AyeAye, Blink W3C Test Autoroller, Olga Sharonova, Raphael Kubo da Costa, AI Code Reviewer, Fredrik Hernqvist, chromium...@chromium.org, Kentaro Hara, Chromium Metrics Reviews, asvitkine...@chromium.org, blink-revie...@chromium.org, android-web...@chromium.org, ashleynewson+w...@chromium.org, jmedle...@chromium.org, kinuko...@chromium.org, blink-revie...@chromium.org, blink-re...@chromium.org, blink-...@chromium.org

Chromium LUCI CQ submitted the change

Change information

Commit message:
Revert "[WebAudio] Implement AudioContext.playbackStats behind a flag"

This reverts commit 46ad5479ba8c6529d1e9a3cdbb38fd6a22997c96.

Reason for revert: Main builders are failing webaudio/AudioContext/audiocontext-playoutstats.html
 test, e.g.:
https://ci.chromium.org/ui/p/chromium/builders/ci/Linux%20Tests/174859/blamelist

Original change's description:
> [WebAudio] Implement AudioContext.playbackStats behind a flag
>
> This CL implements the specification-compliant
> `AudioContext.playbackStats` attribute and the associated
> `AudioPlaybackStats` interface. This new API provides detailed metrics
> regarding audio rendering, including latency extremes and underrun
> (glitch) durations, using standard SI units (seconds).
>
> The legacy `playoutStats` attribute (which uses milliseconds) is
> maintained for backward compatibility in M146 and is scheduled for
> removal in M147. Both APIs are independently gated and tracked via use
> counters.
>
>   - Implemented `AudioContext.playbackStats` and the
>     `AudioPlaybackStats` interface.
>   - Added the `AudioContextPlaybackStats` runtime feature flag and
>     associated use counter.
>   - Refactored `MockPermissionService` to use `mojo::ReceiverSet` to
>     fix unit test instability.
>   - Maintained legacy `playoutStats` with independent test coverage
>     and metrics.
>   - Updated web-exposed API baselines for Blink, WebView, and WPT
>     idlharness.
>
> Bug: 475838360
> Change-Id: I957324965be34a1a1ffddd4964629e782fef8f2e
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7298786
> Reviewed-by: Kent Tamura <tk...@chromium.org>
> Commit-Queue: Hongchan Choi <hong...@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#1579884}
Bug: 475838360
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Change-Id: I735fc8eb5680e21f606b4866236434898bf20b02
Commit-Queue: Rubber Stamper <rubber-...@appspot.gserviceaccount.com>
Auto-Submit: Sergey Poromov <por...@chromium.org>
Reviewed-by: Kent Tamura <tk...@chromium.org>
Commit-Queue: Kent Tamura <tk...@chromium.org>
Bot-Commit: Rubber Stamper <rubber-...@appspot.gserviceaccount.com>
Cr-Commit-Position: refs/heads/main@{#1579909}
Files:
  • M android_webview/test/data/web_tests/webexposed/global-interface-listing-expected.txt
  • M third_party/blink/public/mojom/use_counter/metrics/web_feature.mojom
  • M third_party/blink/renderer/bindings/generated_in_modules.gni
  • M third_party/blink/renderer/bindings/idl_in_modules.gni
  • M third_party/blink/renderer/modules/webaudio/BUILD.gn
  • M third_party/blink/renderer/modules/webaudio/audio_context.cc
  • M third_party/blink/renderer/modules/webaudio/audio_context.h
  • M third_party/blink/renderer/modules/webaudio/audio_context.idl
  • M third_party/blink/renderer/modules/webaudio/audio_context_test.cc
  • D third_party/blink/renderer/modules/webaudio/audio_playback_stats.cc
  • D third_party/blink/renderer/modules/webaudio/audio_playback_stats.h
  • D third_party/blink/renderer/modules/webaudio/audio_playback_stats.idl
  • M third_party/blink/renderer/platform/runtime_enabled_features.json5
  • M third_party/blink/web_tests/VirtualTestSuites
  • M third_party/blink/web_tests/external/wpt/webaudio/idlharness.https.window-expected.txt
  • D third_party/blink/web_tests/external/wpt/webaudio/the-audio-api/the-audiocontext-interface/audiocontext-playbackstats.html
  • R third_party/blink/web_tests/external/wpt/webaudio/the-audio-api/the-audiocontext-interface/audiocontext-playoutstats.html
  • D third_party/blink/web_tests/virtual/webaudio-playback-stats/README.md
  • M third_party/blink/web_tests/webexposed/global-interface-listing-expected.txt
  • M tools/metrics/histograms/metadata/blink/enums.xml
Change size: L
Delta: 20 files changed, 66 insertions(+), 819 deletions(-)
Branch: refs/heads/main
Submit Requirements:
  • requirement satisfiedCode-Review: +1 by Kent Tamura, Bot-Commit+1 by Rubber Stamper
Open in Gerrit
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: merged
Gerrit-Project: chromium/src
Gerrit-Branch: main
Gerrit-Change-Id: I735fc8eb5680e21f606b4866236434898bf20b02
Gerrit-Change-Number: 7543642
Gerrit-PatchSet: 2
open
diffy
satisfied_requirement

Blink W3C Test Autoroller (Gerrit)

unread,
Feb 5, 2026, 4:04:10 AM (6 days ago) Feb 5
to Chromium LUCI CQ, Sergey Poromov, Rubber Stamper, Kent Tamura, Hongchan Choi, AyeAye, Olga Sharonova, Raphael Kubo da Costa, AI Code Reviewer, Fredrik Hernqvist, chromium...@chromium.org, Kentaro Hara, Chromium Metrics Reviews, asvitkine...@chromium.org, blink-revie...@chromium.org, android-web...@chromium.org, ashleynewson+w...@chromium.org, jmedle...@chromium.org, kinuko...@chromium.org, blink-revie...@chromium.org, blink-re...@chromium.org, blink-...@chromium.org

Message from Blink W3C Test Autoroller

The WPT PR for this CL has been merged upstream! https://github.com/web-platform-tests/wpt/pull/57582

Open in Gerrit

Related details

Attention set is empty
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement satisfiedCode-Owners
  • requirement satisfiedCode-Review
  • requirement satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: chromium/src
Gerrit-Branch: main
Gerrit-Change-Id: I735fc8eb5680e21f606b4866236434898bf20b02
Gerrit-Change-Number: 7543642
Gerrit-PatchSet: 2
Gerrit-Owner: Sergey Poromov <por...@chromium.org>
Gerrit-Reviewer: Chromium LUCI CQ <chromiu...@luci-project-accounts.iam.gserviceaccount.com>
Gerrit-Reviewer: Hongchan Choi <hong...@chromium.org>
Gerrit-Reviewer: Kent Tamura <tk...@chromium.org>
Gerrit-Reviewer: Michael Wilson <mjwi...@chromium.org>
Gerrit-Reviewer: Rubber Stamper <rubber-...@appspot.gserviceaccount.com>
Gerrit-Reviewer: Sergey Poromov <por...@chromium.org>
Gerrit-CC: AI Code Reviewer <peep-gen...@system.gserviceaccount.com>
Gerrit-CC: Blink W3C Test Autoroller <blink-w3c-te...@chromium.org>
Gerrit-CC: Chromium Metrics Reviews <chromium-met...@google.com>
Gerrit-CC: Fredrik Hernqvist <fhern...@google.com>
Gerrit-CC: Kentaro Hara <har...@chromium.org>
Gerrit-CC: Olga Sharonova <ol...@google.com>
Gerrit-CC: Raphael Kubo da Costa <ku...@igalia.com>
Gerrit-Comment-Date: Thu, 05 Feb 2026 09:04:03 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
satisfied_requirement
open
diffy
Reply all
Reply to author
Forward
0 new messages