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

android_webview/browser/aw_metrics_service_client.h

Index: android_webview/browser/aw_metrics_service_client.h
diff --git a/android_webview/browser/aw_metrics_service_client.h b/android_webview/browser/aw_metrics_service_client.h
index 29cca691e4b86dfd03acb85b8901e6e1e4257337..551eb9c53f077f259c9a53485d84864164944f2e 100644
--- a/android_webview/browser/aw_metrics_service_client.h
+++ b/android_webview/browser/aw_metrics_service_client.h
@@ -21,6 +21,7 @@ class FilePath;
 
 namespace metrics {
 struct ClientInfo;
+class EnabledStateProvider;
 class MetricsStateManager;
 }
 
@@ -68,6 +69,8 @@ class AwMetricsServiceClient : public metrics::MetricsServiceClient {
   base::TimeDelta GetStandardUploadInterval() override;
 
  private:
+  class AwEnabledStateProvider;
+
   AwMetricsServiceClient();
   ~AwMetricsServiceClient() override;
 
@@ -82,6 +85,7 @@ class AwMetricsServiceClient : public metrics::MetricsServiceClient {
   net::URLRequestContextGetter* request_context_;
   std::unique_ptr<metrics::MetricsStateManager> metrics_state_manager_;
   std::unique_ptr<metrics::MetricsService> metrics_service_;
+  std::unique_ptr<AwEnabledStateProvider> enabled_state_provider_;

[Alexei Svitkine (slow), 2016/05/11 17:16:28]

Nit: How about just making AwMetricsServiceClient implement the interface? Then,
you don't need an extra member/class.

[jwd, 2016/05/11 18:31:38]

Done.

 
   DISALLOW_COPY_AND_ASSIGN(AwMetricsServiceClient);
 };