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

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

Issue 517843005: Remove PrerenderCondition*. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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
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..3021bbd2dd41299a3fe5312a43409c8e67212629 100644
--- a/chrome/browser/prerender/prerender_manager.cc
+++ b/chrome/browser/prerender/prerender_manager.cc
@@ -27,7 +27,6 @@
#include "chrome/browser/net/prediction_options.h"
#include "chrome/browser/predictors/predictor_database.h"
#include "chrome/browser/predictors/predictor_database_factory.h"
-#include "chrome/browser/prerender/prerender_condition.h"
#include "chrome/browser/prerender/prerender_contents.h"
#include "chrome/browser/prerender/prerender_field_trial.h"
#include "chrome/browser/prerender/prerender_final_status.h"
@@ -317,7 +316,6 @@ PrerenderManager::~PrerenderManager() {
void PrerenderManager::Shutdown() {
DestroyAllContents(FINAL_STATUS_MANAGER_SHUTDOWN);
- STLDeleteElements(&prerender_conditions_);
mmenke 2014/09/02 15:11:43 Do we still need the stl_util header?
Bence 2014/09/03 12:51:38 Done.
on_close_web_contents_deleters_.clear();
// Must happen before |profile_| is set to NULL as
// |local_predictor_| accesses it.
@@ -969,10 +967,6 @@ void PrerenderManager::RecordFinalStatusWithMatchCompleteStatus(
final_status);
}
-void PrerenderManager::AddCondition(const PrerenderCondition* condition) {
- prerender_conditions_.push_back(condition);
-}
-
void PrerenderManager::RecordNavigation(const GURL& url) {
DCHECK(CalledOnValidThread());

Powered by Google App Engine
This is Rietveld 408576698