Log metrics for the new Default Search Engine geolocation setting.

chrome/browser/android/search_geolocation/search_geolocation_disclosure_tab_helper.cc

Index: chrome/browser/android/search_geolocation/search_geolocation_disclosure_tab_helper.cc
diff --git a/chrome/browser/android/search_geolocation/search_geolocation_disclosure_tab_helper.cc b/chrome/browser/android/search_geolocation/search_geolocation_disclosure_tab_helper.cc
index 91cd69c0889143249a62735c6e7f3b5205a2f011..3f02fa70b7bf431055d1dae1ce2a2372add2d883 100644
--- a/chrome/browser/android/search_geolocation/search_geolocation_disclosure_tab_helper.cc
+++ b/chrome/browser/android/search_geolocation/search_geolocation_disclosure_tab_helper.cc
@@ -207,6 +207,12 @@ void SearchGeolocationDisclosureTabHelper::RecordPreDisclosureMetrics(
         static_cast<base::HistogramBase::Sample>(status),
         static_cast<base::HistogramBase::Sample>(CONTENT_SETTING_NUM_SETTINGS) +
             1);
+
+    SearchGeolocationService* service =
+        SearchGeolocationService::Factory::GetForBrowserContext(GetProfile());
+    UMA_HISTOGRAM_BOOLEAN("GeolocationDisclosure.PreDisclosureDSESetting",
+                          service->GetDSEGeolocationSetting());
+
     prefs->SetBoolean(prefs::kSearchGeolocationPreDisclosureMetricsRecorded,
                       true);
   }
@@ -226,6 +232,12 @@ void SearchGeolocationDisclosureTabHelper::RecordPostDisclosureMetrics(
         static_cast<base::HistogramBase::Sample>(status),
         static_cast<base::HistogramBase::Sample>(CONTENT_SETTING_NUM_SETTINGS) +
             1);
+
+    SearchGeolocationService* service =
+        SearchGeolocationService::Factory::GetForBrowserContext(GetProfile());
+    UMA_HISTOGRAM_BOOLEAN("GeolocationDisclosure.PostDisclosureDSESetting",
+                          service->GetDSEGeolocationSetting());

[raymes, 2017/04/04 22:36:39]

Sorry my memory on this code isn't the best. Is it possible that if the user
never has their DSE setting set to true, that this will never get executed? In
other words is it possible that:
1) the DSE setting is initialized to false
2) We never show the disclosure
3) The shown count is never incremented, and the user never dismisses the
prompt,
4) We don't enter the codepath above to call RecordPostDisclosureMetrics

Not sure if this affects what you're trying to determine from these metrics
though? Is it hard to test these metrics?

[benwells, 2017/04/05 08:39:49]

Yes that's right, but not a problem. What we're trying to measure with these
metrics is what effect the disclosure has on the setting. The first metric is
pretty much redundant as this code path won't be entered unless the setting is
true, but I'm capturing it anyway for (a) consistency with the other metric and
(b) as a sanity check.

+
     prefs->SetBoolean(prefs::kSearchGeolocationPostDisclosureMetricsRecorded,
                       true);
   }