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

Issue 725103 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Closed: Jun 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Feature



Sign in to add a comment

Collect UMA stats on app/extension permissions used in Public Sessions

Project Member Reported by isandrk@chromium.org, May 22 2017

Issue description

Hey Raj, what are your thoughts on this one? Should we report only installed apps/extensions or also attempted (but rejected) ones?
 
Let's do all of them.
Agreed, we should definitely collect metrics on blocked permissions as well.


Project Member

Comment 3 by bugdroid1@chromium.org, Jun 6 2017

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

commit 5893f141d406fd14f2888e6b094cdd0850668f37
Author: Ivan Sandrk <isandrk@google.com>
Date: Tue Jun 06 14:19:42 2017

Added UMA for extension permissions used in public sessions

Bug:  725103 
Change-Id: I46c6215520d0351f09d4a9fc88786d981aa5b949
Reviewed-on: https://chromium-review.googlesource.com/521602
Reviewed-by: Ivan Šandrk <isandrk@chromium.org>
Reviewed-by: Drew Wilson <atwilson@chromium.org>
Reviewed-by: Ilya Sherman <isherman@chromium.org>
Commit-Queue: Ivan Šandrk <isandrk@chromium.org>
Cr-Commit-Position: refs/heads/master@{#477279}
[modify] https://crrev.com/5893f141d406fd14f2888e6b094cdd0850668f37/chrome/browser/chromeos/extensions/device_local_account_management_policy_provider.cc
[modify] https://crrev.com/5893f141d406fd14f2888e6b094cdd0850668f37/tools/metrics/histograms/histograms.xml

Labels: Merge-Request-60
Status: Fixed (was: Assigned)
Project Member

Comment 5 by sheriffbot@chromium.org, Jun 7 2017

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

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

Comment 6 by bugdroid1@chromium.org, Jun 7 2017

Labels: -merge-approved-60 merge-merged-3112
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/c84f9774b5959c2f80bddb897f3c5beefe992bc3

commit c84f9774b5959c2f80bddb897f3c5beefe992bc3
Author: Ivan Sandrk <isandrk@google.com>
Date: Wed Jun 07 14:29:00 2017

[Merge to M60] Added UMA for extension permissions used in public sessions

Bug:  725103 
Change-Id: I46c6215520d0351f09d4a9fc88786d981aa5b949
Reviewed-on: https://chromium-review.googlesource.com/521602
Reviewed-by: Ivan Šandrk <isandrk@chromium.org>
Reviewed-by: Drew Wilson <atwilson@chromium.org>
Reviewed-by: Ilya Sherman <isherman@chromium.org>
Commit-Queue: Ivan Šandrk <isandrk@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#477279}
Review-Url: https://codereview.chromium.org/2930653002 .
Cr-Commit-Position: refs/branch-heads/3112@{#222}
Cr-Branched-From: b6460e24cf59f429d69de255538d0fc7a425ccf9-refs/heads/master@{#474897}

[modify] https://crrev.com/c84f9774b5959c2f80bddb897f3c5beefe992bc3/chrome/browser/chromeos/extensions/device_local_account_management_policy_provider.cc
[modify] https://crrev.com/c84f9774b5959c2f80bddb897f3c5beefe992bc3/tools/metrics/histograms/histograms.xml

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

Status: Archived (was: Fixed)

Sign in to add a comment