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

Issue 719111 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Closed: May 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Enable Permissions.Action.ProtectedMedia UMA on ChromeOS

Project Member Reported by xhw...@chromium.org, May 6 2017

Issue description

Today we have this enabled on Android because on Android we are using the standard permission bubble. But on ChromeOS, we don't have it since we are using our own platform verification dialog. See issue 454847.
 
Status: Started (was: Assigned)
CL is prepared here: https://chromiumcodereview.appspot.com/2864113002/
Is this still needed given the UMA UI allows us to visualize the metric we want to track via formulas?

https://uma.googleplex.com/p/chrome/timeline_v2/?sid=48f3a236a3c4e746f555c564881bc26f

xhwang@, I realized we could do this only after our chat this morning.
We already have Permissions.Action.ProtectedMedia on Android, which is consistent with all other permissions. I'd like to enable our ChromeOS UMA to use that as well.

Also, currently we don't know how many users actually dismissed the prompt, which will be available in Permissions.Action.ProtectedMedia.

It's a trivial CL, so I feel it's still worth it.
Makes sense - completely agree that knowing how many users are dismissing the prompt is a valuable signal.
Project Member

Comment 5 by bugdroid1@chromium.org, May 9 2017

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

commit caeedfbcb7eb8a9a9a1a262c385af7c49045a864
Author: xhwang <xhwang@chromium.org>
Date: Tue May 09 03:44:59 2017

media: Add Permissions.Action.ProtectedMedia UMA on ChromeOS

BUG= 719111 
TEST=Maually tested

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

[modify] https://crrev.com/caeedfbcb7eb8a9a9a1a262c385af7c49045a864/chrome/browser/media/protected_media_identifier_permission_context.cc

Status: Fixed (was: Started)

Comment 7 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Sign in to add a comment