New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 711348 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug



Sign in to add a comment

Add UMA for whether "Protected Content" is disabled

Project Member Reported by xhw...@chromium.org, Apr 13 2017

Issue description

In  issue 686430 , we added a content settings for users to disable "protected content". We should add a UMA to understand how this is used.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Apr 18 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/604168304f0d95d0fbaf3775b2bd4e1a3ab5dab4

commit 604168304f0d95d0fbaf3775b2bd4e1a3ab5dab4
Author: xhwang <xhwang@chromium.org>
Date: Tue Apr 18 06:53:56 2017

media: Add UMA to record whether encrypted media is enabled

The UMA is recorded when requestMediaKeySystemAccess is called on
non-ClearKey key system but will reported at most once per renderer
process.

BUG= 711348 
TEST=Manually tested

Review-Url: https://codereview.chromium.org/2824643002
Cr-Commit-Position: refs/heads/master@{#465159}

[modify] https://crrev.com/604168304f0d95d0fbaf3775b2bd4e1a3ab5dab4/third_party/WebKit/Source/modules/encryptedmedia/NavigatorRequestMediaKeySystemAccess.cpp
[modify] https://crrev.com/604168304f0d95d0fbaf3775b2bd4e1a3ab5dab4/tools/metrics/histograms/histograms.xml

Comment 2 by xhw...@chromium.org, Apr 20 2017

Labels: Merge-Request-59
The CL has been verified in Canary. Request to merge to M59.
Project Member

Comment 3 by sheriffbot@chromium.org, Apr 20 2017

Labels: -Merge-Request-59 Hotlist-Merge-Approved Merge-Approved-59
Your change meets the bar and is auto-approved for M59. Please go ahead and merge the CL to branch 3071 manually. Please contact milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), gkihumba@(ChromeOS), Abdul Syed@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 4 by bugdroid1@chromium.org, Apr 20 2017

Labels: -merge-approved-59 merge-merged-3071
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/935d9694edaf875ce825a7f8e7e2296c3152cb99

commit 935d9694edaf875ce825a7f8e7e2296c3152cb99
Author: Xiaohan Wang <xhwang@chromium.org>
Date: Thu Apr 20 16:29:58 2017

(merge m59) media: Add UMA to record whether encrypted media is enabled

The UMA is recorded when requestMediaKeySystemAccess is called on
non-ClearKey key system but will reported at most once per renderer
process.

BUG= 711348 
TEST=Manually tested

Review-Url: https://codereview.chromium.org/2824643002
Cr-Commit-Position: refs/heads/master@{#465159}
(cherry picked from commit 604168304f0d95d0fbaf3775b2bd4e1a3ab5dab4)

Review-Url: https://codereview.chromium.org/2834653002 .
Cr-Commit-Position: refs/branch-heads/3071@{#85}
Cr-Branched-From: a106f0abbf69dad349d4aaf4bcc4f5d376dd2377-refs/heads/master@{#464641}

[modify] https://crrev.com/935d9694edaf875ce825a7f8e7e2296c3152cb99/third_party/WebKit/Source/modules/encryptedmedia/NavigatorRequestMediaKeySystemAccess.cpp
[modify] https://crrev.com/935d9694edaf875ce825a7f8e7e2296c3152cb99/tools/metrics/histograms/histograms.xml

Comment 5 by xhw...@chromium.org, Apr 20 2017

Status: Fixed (was: Available)

Sign in to add a comment