tools/metrics/histograms/histograms.xml

Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index a9ea94c896134d471021f918746648ac90b0dc91..d86fedb524e539915f6ebe3d2ea3c347a59e3c43 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -18957,6 +18957,17 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
   </summary>
 </histogram>
 
+<histogram name="Geolocation.Header.PermissionState"
+    enum="GeolocationHeaderPermissionState">
+  <owner>kcarattini@chromium.org</owner>
+  <owner>dominickn@chromium.org</owner>
+  <summary>
+    Counts Geolocation requests for omnibox searches sliced by various
+    permission combinations. The location mode abbreviations indicate one of:
+    High Accuracy, Battery Saving, GPS On or Location Mode Off.
+  </summary>
+</histogram>
+
 <histogram name="Geolocation.InfoBarDelegate.Event"
     enum="GeolocationInfoBarDelegateEvent">
   <obsolete>
@@ -88270,6 +88281,62 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
   <int value="32770" label="PDF"/>
 </enum>
 
+<enum name="GeolocationHeaderPermissionState" type="int">
+  <int value="0" label="Location mode unknown."/>

[Ilya Sherman, 2016/12/05 23:01:41]

Optional nit: I'd omit the periods at the ends of these descriptions, since they
don't make much sense in the UI where these metrics are surfaced.

[pdyson, 2016/12/07 06:13:38]

Done.

+  <int value="1"
+      label="High acc., app allowed, domain allowed, location sent."/>
+  <int value="2"
+      label="High acc., app allowed, domain allowed, location not sent."/>
+  <int value="3" label="High acc., app allowed, domain prompt, location sent."/>
+  <int value="4"
+      label="High acc., app allowed, domain prompt, location not sent."/>
+  <int value="5" label="High acc., app allowed, domain denied."/>
+  <int value="6" label="High acc., app prompt, domain allowed."/>
+  <int value="7" label="High acc., app prompt, domain prompt."/>
+  <int value="8" label="High acc., app prompt, domain denied."/>
+  <int value="9" label="High acc., app denied, domain allowed."/>
+  <int value="10" label="High acc., app denied, domain prompt."/>
+  <int value="11" label="High acc., app denied, domain denied."/>
+  <int value="12"
+      label="Battery sav., app allowed, domain allowed, location sent."/>
+  <int value="13"
+      label="Battery sav., app allowed, domain allowed, location not sent."/>
+  <int value="14"
+      label="Battery sav., app allowed, domain prompt, location sent."/>
+  <int value="15"
+      label="Battery sav., app allowed, domain prompt, location not sent."/>
+  <int value="16" label="Battery sav., app allowed, domain denied."/>
+  <int value="17" label="Battery sav., app prompt, domain allowed."/>
+  <int value="18" label="Battery sav., app prompt, domain prompt."/>
+  <int value="19" label="Battery sav., app prompt, domain denied."/>
+  <int value="20" label="Battery sav., app denied, domain allowed."/>
+  <int value="21" label="Battery sav., app denied, domain prompt."/>
+  <int value="22" label="Battery sav., app denied, domain denied."/>
+  <int value="23"
+      label="GPS only, app allowed, domain allowed, location sent."/>
+  <int value="24"
+      label="GPS only, app allowed, domain allowed, location not sent."/>
+  <int value="25" label="GPS only, app allowed, domain prompt, location sent."/>
+  <int value="26"
+      label="GPS only, app allowed, domain prompt, location not sent."/>
+  <int value="27" label="GPS only, app allowed, domain denied."/>
+  <int value="28" label="GPS only, app prompt, domain allowed."/>
+  <int value="29" label="GPS only, app prompt, domain prompt."/>
+  <int value="30" label="GPS only, app prompt, domain denied."/>
+  <int value="31" label="GPS only, app denied, domain allowed."/>
+  <int value="32" label="GPS only, app denied, domain prompt."/>
+  <int value="33" label="GPS only, app denied, domain denied."/>
+  <int value="34" label="Location off, app allowed, domain allowed."/>
+  <int value="35" label="Location off, app allowed, domain prompt."/>
+  <int value="36" label="Location off, app allowed, domain denied."/>
+  <int value="37" label="Location off, app prompt, domain allowed."/>
+  <int value="38" label="Location off, app prompt, domain prompt."/>
+  <int value="39" label="Location off, app prompt, domain denied."/>
+  <int value="40" label="Location off, app denied, domain allowed."/>
+  <int value="41" label="Location off, app denied, domain prompt."/>
+  <int value="42" label="Location off, app denied, domain denied."/>
+</enum>
+
 <enum name="GeolocationInfoBarDelegateAndroidEvent" type="int">
   <obsolete>
     Deprecated 9/2014, and replaced by PermissionAction.