Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cherry pick PR #2954: Added UMA telemetry for Format Queries #3053

Merged
merged 3 commits into from
Apr 23, 2024

Conversation

cobalt-github-releaser-bot
Copy link
Collaborator

Refer to the original PR: #2954

Modifies the existing class media::FormatSupportQueryMetrics to report latency stats to UMA histograms, and does so on Gold builds. The existing log reporting has been retained for non-Gold builds.

Added basic unit tests to verify that the UMA histograms are being reported correctly.

b/329439820

Change-Id: Ie7f4e2e6c518d1c6c058fcbf62f046b56ea5f298

Modifies the existing class `media::FormatSupportQueryMetrics` to report
latency stats to UMA histograms, and does so on Gold builds. The
existing log reporting has been retained for non-Gold builds.

Added basic unit tests to verify that the UMA histograms are being
reported correctly.

b/329439820

Change-Id: Ie7f4e2e6c518d1c6c058fcbf62f046b56ea5f298
(cherry picked from commit 5e14755)
@cobalt-github-releaser-bot
Copy link
Collaborator Author

Important

There were merge conflicts while cherry picking! Check out cherry-pick-24.lts.1+-2954 and fix the conflicts before proceeding. Check the log at https://github.com/youtube/cobalt/actions/runs/8804513517 for details.

Change-Id: I5b38efc97dce1b5c078f18d11a7bd95ae58d0cc7
@at-ninja at-ninja marked this pull request as ready for review April 23, 2024 18:24
@at-ninja at-ninja enabled auto-merge (squash) April 23, 2024 18:24
….lts.1+-2954

Change-Id: I1980d47780abb72eb861fe985b246a4232b04056
@at-ninja at-ninja merged commit d8e8b05 into 24.lts.1+ Apr 23, 2024
323 checks passed
@at-ninja at-ninja deleted the cherry-pick-24.lts.1+-2954 branch April 23, 2024 20:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants