Don't clear the net predictors prefs on startup

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 160 matching lines @@
   // Cancel pending requests and prevent new ones from being made.
   void Shutdown();
 
   // In some circumstances, for privacy reasons, all results should be
   // discarded.  This method gracefully handles that activity.
   // Destroy all our internal state, which shows what names we've looked up, and
   // how long each has taken, etc. etc.  We also destroy records of suggesses
   // (cache hits etc.).
   void DiscardAllResults();
 
+  // Clears the preferences used by the predictor. Must be called on the UI
+  // thread.
+  void ClearPrefsOnUIThread();

[eroman, 2016/06/25 01:12:53]

Same comment as earlier -- why not put this above the line 167?

[Charlie Harrison, 2016/06/27 18:57:12]

Sorry I missed this one. Done.

+
+  // Calls DiscardAllResults and posts a task to the UI thread to clear prefs.
+  void DiscardAllResultAndClearPrefs();

[eroman, 2016/06/25 01:12:53]

"AllResultAnd" --> "AllResultsAnd"

[Charlie Harrison, 2016/06/27 18:57:12]

The perils of autocomplete :) Done.

+
   // Add hostname(s) to the queue for processing.
   void ResolveList(const std::vector<GURL>& urls,
                    UrlInfo::ResolutionMotivation motivation);
 
   void Resolve(const GURL& url, UrlInfo::ResolutionMotivation motivation);
 
   // Record details of a navigation so that we can preresolve the host name
   // ahead of time the next time the users navigates to the indicated host.
   // Should only be called when urls are distinct, and they should already be
   // canonicalized to not have a path.
@@ skipping 11 matching lines @@
 
   // Construct a ListValue object that contains all the data in the referrers_
   // so that it can be persisted in a pref.
   void SerializeReferrers(base::ListValue* referral_list);
 
   // Process a ListValue that contains all the data from a previous reference
   // list, as constructed by SerializeReferrers(), and add all the identified
   // values into the current referrer list.
   void DeserializeReferrers(const base::ListValue& referral_list);
 
-  void DeserializeReferrersThenDelete(base::ListValue* referral_list);
-
   void DiscardInitialNavigationHistory();
 
   void FinalizeInitializationOnIOThread(
       const std::vector<GURL>& urls_to_prefetch,
-      base::ListValue* referral_list,
+      const base::ListValue* referral_list,
       IOThread* io_thread,
       ProfileIOData* profile_io_data);
 
   // During startup, we learn what the first N urls visited are, and then
   // resolve the associated hosts ASAP during our next startup.
   void LearnAboutInitialNavigation(const GURL& url);
 
   // Renderer bundles up list and sends to this browser API via IPC.
   // TODO(csharrison): Use a GURL vector instead to include port and scheme.
   void DnsPrefetchList(const std::vector<std::string>& hostnames);
@@ skipping 327 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_