Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2323)

Unified Diff: chrome/browser/net/predictor_api.cc

Issue 5921003: Revert 69320 - Migrate HostReferralList and StartupDNSPrefetchList from local... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 10 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/net/predictor_api.h ('k') | chrome/browser/prefs/browser_prefs.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/net/predictor_api.cc
===================================================================
--- chrome/browser/net/predictor_api.cc (revision 69324)
+++ chrome/browser/net/predictor_api.cc (working copy)
@@ -133,9 +133,12 @@
g_browser_process->io_thread()->ChangedToOnTheRecord();
}
+void RegisterPrefs(PrefService* local_state) {
+ local_state->RegisterListPref(prefs::kDnsStartupPrefetchList);
+ local_state->RegisterListPref(prefs::kDnsHostReferralList);
+}
+
void RegisterUserPrefs(PrefService* user_prefs) {
- user_prefs->RegisterListPref(prefs::kDnsPrefetchingStartupList);
- user_prefs->RegisterListPref(prefs::kDnsPrefetchingHostReferralList);
user_prefs->RegisterBooleanPref(prefs::kDnsPrefetchingEnabled, true);
}
@@ -390,22 +393,9 @@
GetPredictedUrlListAtStartup(user_prefs, local_state);
ListValue* referral_list =
- static_cast<ListValue*>(user_prefs->GetMutableList(
- prefs::kDnsPrefetchingHostReferralList)->DeepCopy());
+ static_cast<ListValue*>(
+ local_state->GetMutableList(prefs::kDnsHostReferralList)->DeepCopy());
- // Remove obsolete preferences from local state if necessary.
- bool dns_prefs_version =
- user_prefs->GetInteger(prefs::kMultipleProfilePrefMigration);
- if (dns_prefs_version < 1) {
- // These prefs only need to be registered if they need to be cleared from
- // local state.
- local_state->RegisterListPref(prefs::kDnsStartupPrefetchList);
- local_state->RegisterListPref(prefs::kDnsHostReferralList);
- local_state->ClearPref(prefs::kDnsStartupPrefetchList);
- local_state->ClearPref(prefs::kDnsHostReferralList);
- user_prefs->SetInteger(prefs::kMultipleProfilePrefMigration, 1);
- }
-
g_browser_process->io_thread()->InitNetworkPredictor(
prefetching_enabled, max_dns_queue_delay, max_parallel_resolves, urls,
referral_list, preconnect_enabled);
@@ -472,9 +462,9 @@
BrowserThread::IO,
FROM_HERE,
NewRunnableFunction(SaveDnsPrefetchStateForNextStartupAndTrimOnIOThread,
- prefs->GetMutableList(prefs::kDnsPrefetchingStartupList),
- prefs->GetMutableList(prefs::kDnsPrefetchingHostReferralList),
- &completion));
+ prefs->GetMutableList(prefs::kDnsStartupPrefetchList),
+ prefs->GetMutableList(prefs::kDnsHostReferralList),
+ &completion));
DCHECK(posted);
if (posted)
@@ -490,8 +480,7 @@
// also catch more of the "primary" home pages, since that was (presumably)
// rendered first (and will be rendered first this time too).
ListValue* startup_list =
- user_prefs->GetMutableList(prefs::kDnsPrefetchingStartupList);
-
+ local_state->GetMutableList(prefs::kDnsStartupPrefetchList);
if (startup_list) {
ListValue::iterator it = startup_list->begin();
int format_version = -1;
« no previous file with comments | « chrome/browser/net/predictor_api.h ('k') | chrome/browser/prefs/browser_prefs.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698