Log metrics for the Location Settings Dialog prompt for geolocation.

chrome/browser/geolocation/geolocation_permission_context_android.h

 // Copyright 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_GEOLOCATION_GEOLOCATION_PERMISSION_CONTEXT_ANDROID_H_
 #define CHROME_BROWSER_GEOLOCATION_GEOLOCATION_PERMISSION_CONTEXT_ANDROID_H_
 
 // The flow for geolocation permissions on Android needs to take into account
 // the global geolocation settings so it differs from the desktop one. It
 // works as follows.
@@ skipping 27 matching lines @@
 class InfoBar;
 }
 
 class GURL;
 class PermissionRequestID;
 class PrefRegistrySimple;
 
 class GeolocationPermissionContextAndroid
     : public GeolocationPermissionContext {
  public:
+  enum class LocationSettingsDialogBackOff {

[Ilya Sherman, 2017/04/04 00:04:40]

Please document that this enum is used to back an UMA histogram and should
therefore be treated as append-only.

[benwells, 2017/04/04 08:06:39]

Done.

+    kOneWeek,
+    kOneMonth,
+    kThreeMonths,
+    kMax,

[Ilya Sherman, 2017/04/04 00:04:40]

nit: I'd suggest using "kCount" to mean "the number of items in the enum" and
"kMax" as an alias for the last item in the enum.  So, I'd use "kCount" here.

[benwells, 2017/04/04 08:06:39]

Done.

+  };
+
+  static const char kGeolocationLSDShowDSEMetric[];
+  static const char kGeolocationLSDShowNonDSEMetric[];
+  static const char kGeolocationLSDSuppressDSEMetric[];
+  static const char kGeolocationLSDSuppressNonDSEMetric[];
+  static const char kGeolocationLSDAcceptDSEMetric[];
+  static const char kGeolocationLSDAcceptNonDSEMetric[];
+  static const char kGeolocationLSDDenyDSEMetric[];
+  static const char kGeolocationLSDDenyNonDSEMetric[];
+
   static void RegisterProfilePrefs(PrefRegistrySimple* registry);
 
   explicit GeolocationPermissionContextAndroid(Profile* profile);
   ~GeolocationPermissionContextAndroid() override;
 
  protected:
   // GeolocationPermissionContext:
   ContentSetting GetPermissionStatusInternal(
       content::RenderFrameHost* render_frame_host,
       const GURL& requesting_origin,
@@ skipping 23 matching lines @@
                            const GURL& embedding_origin,
                            const BrowserPermissionCallback& callback,
                            bool persist,
                            ContentSetting content_setting) override;
   PermissionResult UpdatePermissionStatusWithDeviceStatus(
       PermissionResult result,
       const GURL& requesting_origin,
       const GURL& embedding_origin) const override;
 
   // Functions to handle back off for showing the Location Settings Dialog.
-  std::string GetLocationSettingsBackOffLevelPref(
-      const GURL& requesting_origin) const;
-  std::string GetLocationSettingsNextShowPref(
-      const GURL& requesting_origin) const;
-  bool IsInLocationSettingsBackOff(const GURL& requesting_origin) const;
-  void ResetLocationSettingsBackOff(const GURL& requesting_origin);
-  void UpdateLocationSettingsBackOff(const GURL& requesting_origin);
+  std::string GetLocationSettingsBackOffLevelPref(bool is_dse) const;
+  std::string GetLocationSettingsNextShowPref(bool is_dse) const;
+  bool IsInLocationSettingsBackOff(bool is_dse) const;
+  void ResetLocationSettingsBackOff(bool is_dse);
+  void UpdateLocationSettingsBackOff(bool is_dse);
+  LocationSettingsDialogBackOff LocationSettingsBackOffLevel(bool is_dse) const;
 
   // Returns whether location access is possible for the given origin. Ignores
   // Location Settings Dialog backoff, as the backoff is ignored if the user
   // will be prompted for permission.
   bool IsLocationAccessPossible(content::WebContents* web_contents,
                                 const GURL& requesting_origin,
                                 bool user_gesture);
 
   bool IsRequestingOriginDSE(const GURL& requesting_origin) const;
 
@@ skipping 38 matching lines @@
   infobars::InfoBar* permission_update_infobar_;
 
   // Must be the last member, to ensure that it will be destroyed first, which
   // will invalidate weak pointers.
   base::WeakPtrFactory<GeolocationPermissionContextAndroid> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(GeolocationPermissionContextAndroid);
 };
 
 #endif  // CHROME_BROWSER_GEOLOCATION_GEOLOCATION_PERMISSION_CONTEXT_ANDROID_H_