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

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

Issue 459953002: Migrate geolocation permissions to the new common permission class. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 6 years, 3 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 | « content/public/browser/content_browser_client.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 10b2c103490a166f4d8661a89914401ea7683ee4..7f199c56a19713a4234ab23a6549fe3c8500e1b1 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -9272,12 +9272,20 @@ Therefore, the affected-histogram name has to have at least one dot in it.
<histogram name="Geolocation.InfoBarDelegate.Event"
enum="GeolocationInfoBarDelegateEvent">
+ <obsolete>
+ Deprecated 9/2014, and replaced by
+ ContentSettingsPermissionActions_Geolocation.
Mark P 2014/09/24 17:59:38 missing "." between Settings and Permission ditto
Miguel Garcia 2014/09/29 14:24:58 Done.
+ </obsolete>
<owner>mvanouwerkerk@chromium.org</owner>
<summary>Events in GeolocationInfoBarDelegate.</summary>
</histogram>
<histogram name="Geolocation.InfoBarDelegateAndroid.Event"
enum="GeolocationInfoBarDelegateAndroidEvent">
+ <obsolete>
+ Deprecated 9/2014, and replaced by
+ ContentSettingsPermissionActions_Geolocation.
+ </obsolete>
<owner>mvanouwerkerk@chromium.org</owner>
<summary>Events in GeolocationInfoBarDelegateAndroid.</summary>
</histogram>
@@ -43509,6 +43517,9 @@ Therefore, the affected-histogram name has to have at least one dot in it.
</enum>
<enum name="GeolocationInfoBarDelegateAndroidEvent" type="int">
+ <obsolete>
+ Deprecated 9/2014, and replaced by PermissionAction.
+ </obsolete>
<int value="0" label="User allowed the page to use geolocation">
For the Android platform the count for this event should be exactly the same
as the corresponding event in the GeolocationInfoBarDelegateEvent enum.
@@ -43517,6 +43528,9 @@ Therefore, the affected-histogram name has to have at least one dot in it.
</enum>
<enum name="GeolocationInfoBarDelegateEvent" type="int">
+ <obsolete>
+ Deprecated 9/2014, and replaced by PermissionAction.
+ </obsolete>
<int value="0" label="The bar was created"/>
<int value="1" label="User allowed use of geolocation"/>
<int value="2" label="User denied use of geolocation"/>
@@ -55269,6 +55283,7 @@ To add a new entry, add it with any value and run test to compute valid value.
<suffix name="MidiSysEx" label="Midi SysEx permsision actions"/>
<suffix name="PushMessaging" label="Push messaging permission actions"/>
<suffix name="Notifications" label="Notification permission actions"/>
+ <suffix name="Geolocation" label="Geolocation permission actions"/>
<affected-histogram name="ContentSettings.PermissionActions"/>
</histogram_suffixes>
« no previous file with comments | « content/public/browser/content_browser_client.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698