Use lossy prefs in the net predictor, and update them more frequently

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/feature_list.h"
+#include "chrome/browser/chromeos/profiles/profile_helper.h"
 #include "chrome/browser/net/predictor.h"
 #include "chrome/browser/profiles/profile.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 {
 
 namespace {
@@ skipping 28 matching lines @@
 }
 
 void PredictorTabHelper::DidStartNavigationToPendingEntry(
     const GURL& url,
     content::NavigationController::ReloadType reload_type) {
   // The standard way to preconnect based on navigation.
   PreconnectUrl(url);
   predicted_from_pending_entry_ = true;
 }
 
+void PredictorTabHelper::DocumentOnLoadCompletedInMainFrame() {
+  Profile* profile =
+      Profile::FromBrowserContext(web_contents()->GetBrowserContext());
+  Predictor* predictor(profile->GetNetworkPredictor());

[Bernhard Bauer, 2016/07/29 11:07:53]

Nit: I would use assignment here to initialize |predictor|.

[Charlie Harrison, 2016/07/29 13:42:13]

Done.

+#if defined(OS_CHROMEOS)
+  if (chromeos::ProfileHelper::IsSigninProfile(profile))
+    return;
+#endif
+  if (predictor)
+    predictor->SaveStateForNextStartup();
+}
+
 void PredictorTabHelper::PreconnectUrl(const GURL& url) {
   Profile* profile =
       Profile::FromBrowserContext(web_contents()->GetBrowserContext());
-  chrome_browser_net::Predictor* predictor = profile->GetNetworkPredictor();
+  Predictor* predictor(profile->GetNetworkPredictor());
   if (predictor && url.SchemeIsHTTPOrHTTPS())
     predictor->PreconnectUrlAndSubresources(url, GURL());
 }
 
 }  // namespace chrome_browser_net