Componentize renderer side of DNS prefetching.

chrome/browser/net/predictor.h

 // Copyright (c) 2012 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.
 
 // A Predictor object is instantiated once in the browser process, and manages
 // both preresolution of hostnames, as well as TCP/IP preconnection to expected
 // subresources.
 // Most hostname lists are provided by the renderer processes, and include URLs
 // that *might* be used in the near future by the browsing user.  One goal of
 // this class is to cause the underlying DNS structure to lookup a hostname
@@ skipping 15 matching lines @@
 #include <vector>
 
 #include "base/gtest_prod_util.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
 #include "chrome/browser/net/prediction_options.h"
 #include "chrome/browser/net/referrer.h"
 #include "chrome/browser/net/spdyproxy/proxy_advisor.h"
 #include "chrome/browser/net/timed_cache.h"
 #include "chrome/browser/net/url_info.h"
-#include "chrome/common/net/predictor_common.h"
+#include "components/predictor/common/predictor_common.h"

[Nico, 2014/10/28 20:07:19]

is "predictor" the best name if this is just in components/? will dns be the
only thing a component will predict?

[gunsch, 2014/10/28 20:47:32]

Sounds like a +1 to blundell's idea, so renaming the component "dns_prefetch"

 #include "net/base/host_port_pair.h"
 
 class IOThread;
 class PrefService;
 class Profile;
 class ProfileIOData;
 
 namespace base {
 class ListValue;
 class WaitableEvent;
 }
 
 namespace net {
 class HostResolver;
 class SSLConfigService;
 class ProxyService;
 class TransportSecurityState;
 class URLRequestContextGetter;
 }
 
 namespace user_prefs {
 class PrefRegistrySyncable;
 }
 
 namespace chrome_browser_net {
 
-typedef chrome_common_net::UrlList UrlList;
-typedef chrome_common_net::NameList NameList;
+typedef predictor::UrlList UrlList;
+typedef predictor::NameList NameList;
 typedef std::map<GURL, UrlInfo> Results;
 
 // An observer for testing.
 class PredictorObserver {
  public:
   virtual ~PredictorObserver() {}
 
   virtual void OnPreconnectUrl(const GURL& original_url,
                                const GURL& first_party_for_cookies,
                                UrlInfo::ResolutionMotivation motivation,
@@ skipping 554 matching lines @@
 
  private:
   // These member functions return True for unittests.
   bool CanPrefetchAndPrerender() const override;
   bool CanPreresolveAndPreconnect() const override;
 };
 
 }  // namespace chrome_browser_net
 
 #endif  // CHROME_BROWSER_NET_PREDICTOR_H_