WebView: Remove URLRequestContextGetter from AwMetricsServiceClient

It's been unused since commit a5febff9 and null since
6bd4db44.

BUG=546754

Change-Id: Idbffc047f67d3c424a2d6557e50dac8be9d9cee4
Reviewed-on: https://chromium-review.googlesource.com/1236695Reviewed-by: Bo <boliu@chromium.org>
Commit-Queue: Paul Miller <paulmiller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#592971}

android_webview/browser/aw_browser_main_parts.cc

@@ -223,8 +223,7 @@ int AwBrowserMainParts::PreCreateThreads() {
 
   browser_policy_connector_ = std::make_unique<AwBrowserPolicyConnector>();
   pref_service_ = CreatePrefService(browser_policy_connector_.get());
-  AwMetricsServiceClient::GetInstance()->Initialize(pref_service_.get(),
-                                                    nullptr);
+  AwMetricsServiceClient::GetInstance()->Initialize(pref_service_.get());
   aw_field_trial_creator_.SetUpFieldTrials(pref_service_.get());
 
   return service_manager::RESULT_CODE_NORMAL_EXIT;

android_webview/browser/aw_metrics_service_client.cc

@@ -133,15 +133,11 @@ std::string AwMetricsServiceClient::GetClientId() {
   return g_client_id.Get();
 }
 
-void AwMetricsServiceClient::Initialize(
-    PrefService* pref_service,
-    net::URLRequestContextGetter* request_context) {
+void AwMetricsServiceClient::Initialize(PrefService* pref_service) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
 
   DCHECK(pref_service_ == nullptr);  // Initialize should only happen once.
-  DCHECK(request_context_ == nullptr);
   pref_service_ = pref_service;
-  request_context_ = request_context;
 
   metrics_state_manager_ = metrics::MetricsStateManager::Create(
       pref_service_, this, base::string16(),
@@ -273,7 +269,6 @@ base::TimeDelta AwMetricsServiceClient::GetStandardUploadInterval() {
 
 AwMetricsServiceClient::AwMetricsServiceClient()
     : pref_service_(nullptr),
-      request_context_(nullptr),
       consent_(false),
       in_sample_(false) {}
 

android_webview/browser/aw_metrics_service_client.h

@@ -21,10 +21,6 @@ namespace base {
 class FilePath;
 }
 
-namespace net {
-class URLRequestContextGetter;
-}
-
 namespace metrics {
 class MetricsStateManager;
 }
@@ -50,8 +46,7 @@ class AwMetricsServiceClient : public metrics::MetricsServiceClient,
   // Return the cached client id.
   static std::string GetClientId();
 
-  void Initialize(PrefService* pref_service,
-                  net::URLRequestContextGetter* request_context);
+  void Initialize(PrefService* pref_service);
 
   std::unique_ptr<const base::FieldTrial::EntropyProvider>
   CreateLowEntropyProvider();
@@ -92,7 +87,6 @@ class AwMetricsServiceClient : public metrics::MetricsServiceClient,
   std::unique_ptr<metrics::MetricsStateManager> metrics_state_manager_;
   std::unique_ptr<metrics::MetricsService> metrics_service_;
   PrefService* pref_service_;
-  net::URLRequestContextGetter* request_context_;
   bool consent_;    // = (user has consented) && !(app has opted out)
   bool in_sample_;  // Is this client enabled by sampling?