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

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

Issue 11551003: Change multi-prerender API to include per launcher slots. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: clear to land Created 8 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/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 4dfcba87f08884d03fd3809eca62c489d3a5a48d..ebc042cfac2c358acae4a51eb9a58b0589b7543c 100644
--- a/chrome/browser/prerender/prerender_manager.cc
+++ b/chrome/browser/prerender/prerender_manager.cc
@@ -99,7 +99,6 @@ const int kHistoryLength = 100;
bool NeedMatchCompleteDummyForFinalStatus(FinalStatus final_status) {
return final_status != FINAL_STATUS_USED &&
final_status != FINAL_STATUS_TIMED_OUT &&
- final_status != FINAL_STATUS_EVICTED &&
final_status != FINAL_STATUS_MANAGER_SHUTDOWN &&
final_status != FINAL_STATUS_APP_TERMINATING &&
final_status != FINAL_STATUS_WINDOW_OPENER &&
@@ -209,7 +208,8 @@ PrerenderManager::PrerenderManager(Profile* profile,
// Certain experiments override our default config_ values.
switch (PrerenderManager::GetMode()) {
case PrerenderManager::PRERENDER_MODE_EXPERIMENT_MULTI_PRERENDER_GROUP:
- config_.max_concurrency = 3;
+ config_.max_link_concurrency = 4;
+ config_.max_link_concurrency_per_launcher = 2;
break;
case PrerenderManager::PRERENDER_MODE_EXPERIMENT_15MIN_TTL_GROUP:
config_.time_to_live = base::TimeDelta::FromMinutes(15);
@@ -1046,14 +1046,8 @@ PrerenderHandle* PrerenderManager::AddPrerender(
DCHECK(IsControlGroup(experiment) ||
prerender_contents->prerendering_has_started());
- while (active_prerenders_.size() > config_.max_concurrency) {
- prerender_contents = active_prerenders_.front()->contents();
- DCHECK(prerender_contents);
- prerender_contents->Destroy(FINAL_STATUS_EVICTED);
- }
-
- histograms_->RecordConcurrency(active_prerenders_.size(),
- config_.max_concurrency);
+ if (GetMode() == PRERENDER_MODE_EXPERIMENT_MULTI_PRERENDER_GROUP)
+ histograms_->RecordConcurrency(active_prerenders_.size());
StartSchedulingPeriodicCleanups();
return prerender_handle;
« 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