Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(12)

Unified Diff: tools/metrics/histograms/histograms.xml

Side-by-side diff isn't available for this file because of its large size.
Issue 2684913002: Add UMA for recording embargo reasons and autoblocker interactions. (Closed)
Patch Set: Rebase forever Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
Download patch
« no previous file with comments | « chrome/browser/permissions/permission_uma_util.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index 539e6baf27644429b42f629fdb16b33eb44680b4..77907f10f22a15eba5bb8a4390c11ff4f65cc13d 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -46045,25 +46045,18 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</summary>
</histogram>
-<histogram name="Permissions.AutoBlocker.SafeBrowsingResponse"
- enum="SafeBrowsingResponse">
+<histogram name="Permissions.AutoBlocker.EmbargoStatus"
+ enum="PermissionEmbargoStatus">
<owner>dominickn@chromium.org</owner>
<owner>kcarattini@chromium.org</owner>
<summary>
- Tracks the response (if received) from Safe Browsing when the API blacklist
- is queried for an (origin, permission) pair. The response could be that the
- origin was blacklisted, not blacklisted, or Safe Browsing timed out before a
- response was received.
- </summary>
-</histogram>
-
-<histogram name="Permissions.AutoBlocker.SafeBrowsingResponseTime" units="ms">
- <owner>dominickn@chromium.org</owner>
- <owner>kcarattini@chromium.org</owner>
- <summary>
- Records the elapsed time between the client sending a request to Safe
- Browsing and receiving a result, or the maximum wait time is exceeded and
- Safe Browsing is deemed to have timed out.
+ Tracks the reason that an (origin, permission) pair has been placed under
+ embargo (blocked from making requests for that permission for a period of
+ time). This is triggered when a permission is placed under embargo for
+ blacklisting (once safe browsing has returned a result), when a permission
+ prompt has been shown and it is dismissed, or when a permission prompt has
+ been shown and the user either allow/blocked/ignored. In the latter case, it
+ is recorded when the permission request is destroyed.
</summary>
</histogram>
@@ -102337,6 +102330,12 @@ value.
<int value="4" label="REVOKED"/>
</enum>
+<enum name="PermissionEmbargoStatus" type="int">
+ <int value="0" label="NOT_EMBARGOED"/>
+ <int value="1" label="BLACKLISTED"/>
+ <int value="2" label="REPEATED_DISMISSALS"/>
+</enum>
+
<enum name="PermissionRequestType" type="int">
<int value="0" label="PERMISSION_BUBBLE_UNKNOWN"/>
<int value="1" label="PERMISSION_BUBBLE_MULTIPLE"/>
« no previous file with comments | « chrome/browser/permissions/permission_uma_util.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698