Change net's predictor_tab_helper to use the new content Navigation API

chrome/browser/net/predictor_tab_helper.cc

 // Copyright 2013 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/net/predictor_tab_helper.h"
 
 #include "base/command_line.h"
 #include "chrome/browser/net/predictor.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/url_constants.h"
+#include "content/public/browser/navigation_handle.h"
 
 DEFINE_WEB_CONTENTS_USER_DATA_KEY(chrome_browser_net::PredictorTabHelper);
 
 namespace chrome_browser_net {
 
 PredictorTabHelper::PredictorTabHelper(content::WebContents* web_contents)
     : content::WebContentsObserver(web_contents) {
 }
 
 PredictorTabHelper::~PredictorTabHelper() {
 }
 
-void PredictorTabHelper::DidStartNavigationToPendingEntry(
-    const GURL& url,
-    content::NavigationController::ReloadType reload_type) {
+void PredictorTabHelper::DidStartNavigation(
+    content::NavigationHandle* navigation_handle) {
   Profile* profile =
       Profile::FromBrowserContext(web_contents()->GetBrowserContext());
   chrome_browser_net::Predictor* predictor = profile->GetNetworkPredictor();
   if (!predictor)
     return;
+
+  const GURL& url = navigation_handle->GetURL();
   if (url.SchemeIs(url::kHttpScheme) || url.SchemeIs(url::kHttpsScheme))

[mmenke, 2016/03/17 18:52:15]

While we're here, maybe get rid of url local, and use
navigation_handle->GetURL().SchemeIsHTTPOrHTTPS()?

[Charlie Harrison, 2016/03/17 21:22:41]

ACK

     predictor->PreconnectUrlAndSubresources(url, GURL());
 }
 
 }  // namespace chrome_browser_net