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

Unified Diff: chrome/browser/prerender/prerender_manager.cc

Issue 519583005: Remove PrerenderManager.enabled_ and related members. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Nits. Created 6 years, 3 months 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/prerender/prerender_manager.h ('k') | chrome/browser/prerender/prerender_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/prerender/prerender_manager.cc
diff --git a/chrome/browser/prerender/prerender_manager.cc b/chrome/browser/prerender/prerender_manager.cc
index 3627820ad4329d80faf983d13e4a6647300e7aeb..b6981df749e76d897e3a9ce49f3b8e275ae1d6f5 100644
--- a/chrome/browser/prerender/prerender_manager.cc
+++ b/chrome/browser/prerender/prerender_manager.cc
@@ -238,8 +238,7 @@ struct PrerenderManager::NavigationRecord {
PrerenderManager::PrerenderManager(Profile* profile,
PrerenderTracker* prerender_tracker)
- : enabled_(true),
- profile_(profile),
+ : profile_(profile),
prerender_tracker_(prerender_tracker),
prerender_contents_factory_(PrerenderContents::CreateFactory()),
last_prerender_start_time_(GetCurrentTimeTicks() -
@@ -741,11 +740,6 @@ void PrerenderManager::RecordPerceivedPageLoadTime(
}
}
-void PrerenderManager::set_enabled(bool enabled) {
- DCHECK(CalledOnValidThread());
- enabled_ = enabled;
-}
-
// static
PrerenderManager::PrerenderManagerMode PrerenderManager::GetMode() {
return mode_;
@@ -933,7 +927,7 @@ base::DictionaryValue* PrerenderManager::GetAsValue() const {
base::DictionaryValue* dict_value = new base::DictionaryValue();
dict_value->Set("history", prerender_history_->GetEntriesAsValue());
dict_value->Set("active", GetActivePrerendersAsValue());
- dict_value->SetBoolean("enabled", enabled_);
+ dict_value->SetBoolean("enabled", IsEnabled());
dict_value->SetBoolean("omnibox_enabled", IsOmniboxEnabled(profile_));
// If prerender is disabled via a flag this method is not even called.
std::string enabled_note;
@@ -1861,8 +1855,6 @@ void PrerenderManager::RecordNetworkBytes(Origin origin,
bool PrerenderManager::IsEnabled() const {
DCHECK(CalledOnValidThread());
- if (!enabled_)
- return false;
return chrome_browser_net::CanPrefetchAndPrerenderUI(profile_->GetPrefs());
}
« no previous file with comments | « chrome/browser/prerender/prerender_manager.h ('k') | chrome/browser/prerender/prerender_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698