[M144] media: Report multi-GPU histograms when MediaFoundation playback starts [chromium/src : refs/branch-heads/7559]

0 views
Skip to first unread message

Chrome Cherry Picker (Gerrit)

unread,
Dec 25, 2025, 1:01:20 PM (2 days ago) Dec 25
to Sangbaek Park, Chromium Metrics Reviews, AyeAye, asvitkine...@chromium.org, feature-me...@chromium.org

Chrome Cherry Picker voted Auto-Submit+1

Auto-Submit+1
Open in Gerrit

Related details

Attention set is empty
Submit Requirements:
  • requirement satisfiedCode-Owners
  • requirement is not satisfiedCode-Review
  • requirement satisfiedLint
  • 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: refs/branch-heads/7559
Gerrit-Change-Id: I48f4fda7d60f0a2ab638375c5addf0246c4aaa4e
Gerrit-Change-Number: 7326656
Gerrit-PatchSet: 2
Gerrit-Reviewer: Chrome Cherry Picker <chrome-che...@chops-service-accounts.iam.gserviceaccount.com>
Gerrit-CC: Chromium Metrics Reviews <chromium-met...@google.com>
Gerrit-CC: Sangbaek Park <sangba...@chromium.org>
Gerrit-Comment-Date: Thu, 25 Dec 2025 18:01:13 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
unsatisfied_requirement
open
diffy

Rubber Stamper (Gerrit)

unread,
Dec 25, 2025, 1:02:22 PM (2 days ago) Dec 25
to Sangbaek Park, Chrome Cherry Picker, Chromium Metrics Reviews, AyeAye, asvitkine...@chromium.org, feature-me...@chromium.org

Rubber Stamper voted

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

Related details

Attention set is empty
Submit Requirements:
  • requirement satisfiedCode-Owners
  • requirement satisfiedCode-Review
  • requirement satisfiedLint
  • 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: refs/branch-heads/7559
Gerrit-Change-Id: I48f4fda7d60f0a2ab638375c5addf0246c4aaa4e
Gerrit-Change-Number: 7326656
Gerrit-PatchSet: 2
Gerrit-Reviewer: Chrome Cherry Picker <chrome-che...@chops-service-accounts.iam.gserviceaccount.com>
Gerrit-Reviewer: Rubber Stamper <rubber-...@appspot.gserviceaccount.com>
Gerrit-CC: Chromium Metrics Reviews <chromium-met...@google.com>
Gerrit-CC: Sangbaek Park <sangba...@chromium.org>
Gerrit-Comment-Date: Thu, 25 Dec 2025 18:02:08 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy

Chromium LUCI CQ (Gerrit)

unread,
Dec 25, 2025, 3:02:24 PM (2 days ago) Dec 25
to Sangbaek Park, Chrome Cherry Picker, Rubber Stamper, Chromium Metrics Reviews, AyeAye, asvitkine...@chromium.org, feature-me...@chromium.org

Chromium LUCI CQ submitted the change

Change information

Commit message:
[M144] media: Report multi-GPU histograms when MediaFoundation playback starts

Original change's description:
> media: Report multi-GPU histograms when MediaFoundation playback starts
>
> - Report multi-GPU UMAs only once when the virtual video window size is
> the correct one (the output rect is larger than default size) since the
> window handle is required to determine where the video is displayed.
> - These histograms are required for further investigation on the high
> volume of "hardware-context-reset" cases with PlayReady SL3000 on
> Chrome Windows.
>
> Tests:
> - Manually verified the histograms only reported once.
> - See the histogram example (tested 4 playbacks):
> http://shortn/_jDZxSVXsZH
>
> Bug: b:454139099, b:454691291, 469759460
> Change-Id: I48f4fda7d60f0a2ab638375c5addf0246c4aaa4e
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7270236
> Reviewed-by: Evan Liu <ev...@google.com>
> Reviewed-by: Dale Curtis <dalec...@chromium.org>
> Commit-Queue: Sangbaek Park <sangba...@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#1560224}

(cherry picked from commit acd28a155b356ec7b1f39576f30a2776c86a582b)
Bug: 470062916,b:454139099,b:454691291,469759460
Change-Id: I48f4fda7d60f0a2ab638375c5addf0246c4aaa4e
Commit-Queue: Rubber Stamper <rubber-...@appspot.gserviceaccount.com>
Bot-Commit: Rubber Stamper <rubber-...@appspot.gserviceaccount.com>
Cr-Commit-Position: refs/branch-heads/7559@{#2238}
Cr-Branched-From: 223dfbac1c7542a06b422390d954afe5b560b607-refs/heads/main@{#1552494}
Files:
  • M media/renderers/win/media_foundation_renderer.cc
  • M media/renderers/win/media_foundation_renderer.h
  • M tools/metrics/histograms/metadata/media/histograms.xml
Change size: M
Delta: 3 files changed, 117 insertions(+), 1 deletion(-)
Branch: refs/branch-heads/7559
Submit Requirements:
  • requirement satisfiedCode-Review: 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: refs/branch-heads/7559
Gerrit-Change-Id: I48f4fda7d60f0a2ab638375c5addf0246c4aaa4e
Gerrit-Change-Number: 7326656
Gerrit-PatchSet: 3
Gerrit-Reviewer: Chrome Cherry Picker <chrome-che...@chops-service-accounts.iam.gserviceaccount.com>
Gerrit-Reviewer: Chromium LUCI CQ <chromiu...@luci-project-accounts.iam.gserviceaccount.com>
open
diffy
satisfied_requirement
Reply all
Reply to author
Forward
0 new messages