Collect UMA stats on app/extension permissions used in Public Sessions |
|||||
Issue descriptionHey Raj, what are your thoughts on this one? Should we report only installed apps/extensions or also attempted (but rejected) ones?
,
May 23 2017
Agreed, we should definitely collect metrics on blocked permissions as well.
,
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
,
Jun 6 2017
,
Jun 7 2017
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
,
Jun 7 2017
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
,
Jan 22 2018
|
|||||
►
Sign in to add a comment |
|||||
Comment 1 by atwilson@chromium.org
, May 23 2017