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

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

Issue 516443002: Remove NETWORK_PREDICTION_UNSET. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add DCHECK. Created 6 years, 4 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/prefetch/prefetch.cc ('k') | no next file » | 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 ea4bba111536acb9d665c6cc904640be56e7e3c8..3627820ad4329d80faf983d13e4a6647300e7aeb 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_(profile && profile->GetPrefs() &&
- profile->GetPrefs()->GetBoolean(prefs::kNetworkPredictionEnabled)),
+ : enabled_(true),
profile_(profile),
prerender_tracker_(prerender_tracker),
prerender_contents_factory_(PrerenderContents::CreateFactory()),
@@ -1862,24 +1861,9 @@ void PrerenderManager::RecordNetworkBytes(Origin origin,
bool PrerenderManager::IsEnabled() const {
DCHECK(CalledOnValidThread());
- // TODO(bnc): remove conditional as per crbug.com/334602.
- if (profile_ && profile_->GetPrefs() &&
- profile_->GetPrefs()->GetInteger(prefs::kNetworkPredictionOptions) !=
- chrome_browser_net::NETWORK_PREDICTION_UNSET) {
- return chrome_browser_net::CanPrefetchAndPrerenderUI(profile_->GetPrefs());
- }
- // TODO(bnc): remove rest of method as per crbug.com/334602.
if (!enabled_)
return false;
- for (std::list<const PrerenderCondition*>::const_iterator it =
- prerender_conditions_.begin();
- it != prerender_conditions_.end();
- ++it) {
- const PrerenderCondition* condition = *it;
- if (!condition->CanPrerender())
- return false;
- }
- return true;
+ return chrome_browser_net::CanPrefetchAndPrerenderUI(profile_->GetPrefs());
}
void PrerenderManager::AddProfileNetworkBytesIfEnabled(int64 bytes) {
« no previous file with comments | « chrome/browser/prefetch/prefetch.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698