Splitting the concept of UMA consent, and should UMA report.

blimp/engine/app/blimp_metrics_service_client.cc

Index: blimp/engine/app/blimp_metrics_service_client.cc
diff --git a/blimp/engine/app/blimp_metrics_service_client.cc b/blimp/engine/app/blimp_metrics_service_client.cc
index 20db13dee244073695d7bdd7c555ffa2578a627e..060ff9e32097d91b706279914f45d8aa9f13e337 100644
--- a/blimp/engine/app/blimp_metrics_service_client.cc
+++ b/blimp/engine/app/blimp_metrics_service_client.cc
@@ -31,13 +31,6 @@ namespace {
 // metrics service.
 const int kStandardUploadIntervalMinutes = 30;
 
-// Returns if the MetricsService should be recording metrics information for
-// the client. Always true.
-// This callback is required by MetricsStateManager::Create.
-bool IsReportingEnabled() {
-  return true;
-}
-
 // Store/LoadClientInfo allows Windows Chrome to back up ClientInfo.
 // Both are no-ops for Blimp.
 // These callbacks are required by MetricsStateManager::Create.
@@ -55,8 +48,8 @@ BlimpMetricsServiceClient::BlimpMetricsServiceClient(
   request_context_getter_ = request_context_getter;
 
   metrics_state_manager_ = metrics::MetricsStateManager::Create(
-      pref_service, base::Bind(&IsReportingEnabled),
-      base::Bind(&StoreClientInfo), base::Bind(&LoadClientInfo));
+      pref_service, this, base::Bind(&StoreClientInfo),
+      base::Bind(&LoadClientInfo));
 
   // Metrics state manager created while other class instances exist.
   // Sign of multiple initializations.
@@ -163,5 +156,9 @@ BlimpMetricsServiceClient::GetDefaultOptIn() {
   return OPT_IN;
 }
 
+bool BlimpMetricsServiceClient::IsConsentGiven() {
+  return true;
+}
+
 }  // namespace engine
 }  // namespace blimp