predictors: Remove ResourcePrefetchPredictorConfig from ResourcePrefetcher.

chrome/browser/predictors/resource_prefetcher_manager.cc

 // Copyright 2014 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.
 
 #include "chrome/browser/predictors/resource_prefetcher_manager.h"
 
 #include <utility>
 
 #include "base/bind.h"
 #include "base/memory/ptr_util.h"
@@ skipping 42 matching lines @@
 void ResourcePrefetcherManager::MaybeAddPrefetch(
     const GURL& main_frame_url,
     const std::vector<GURL>& urls) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::IO);
 
   // Don't add a duplicate prefetch for the same host.
   std::string key = main_frame_url.host();
   if (base::ContainsKey(prefetcher_map_, key))
     return;
 
-  auto prefetcher =
-      base::MakeUnique<ResourcePrefetcher>(this, config_, main_frame_url, urls);

[alexilin, 2017/04/27 16:39:15]

Does it make a sense to remove ResourcePrefetchPredictorConfig from the
ResourcePrefetcherManager as well? 

[Benoit L, 2017/04/27 17:14:42]

This will be merged into the policy part, that will need knowledge of the
configuration.

+  auto prefetcher = base::MakeUnique<ResourcePrefetcher>(
+      this, config_.max_prefetches_inflight_per_navigation,
+      config_.max_prefetches_inflight_per_host_per_navigation, main_frame_url,
+      urls);
   prefetcher->Start();
   prefetcher_map_[key] = std::move(prefetcher);
 }
 
 void ResourcePrefetcherManager::MaybeRemovePrefetch(
     const GURL& main_frame_url) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::IO);
 
   std::string key = main_frame_url.host();
   auto it = prefetcher_map_.find(key);
@@ skipping 21 matching lines @@
   prefetcher_map_.erase(it);
 }
 
 net::URLRequestContext* ResourcePrefetcherManager::GetURLRequestContext() {
   DCHECK_CURRENTLY_ON(content::BrowserThread::IO);
 
   return context_getter_->GetURLRequestContext();
 }
 
 }  // namespace predictors