From f7ece85cb9909c76eb344f2462bb7b6c55973d3f Mon Sep 17 00:00:00 2001 From: "mmenke@chromium.org" Date: Sat, 6 Aug 2011 03:19:38 +0000 Subject: Fix prerenders in the delete list not being returned by PrerenderManager::GetAsValue(). NetInternalsTest.NetInternalsPrerenderView expects to always see the prerender as active or in the history, once started. BUG=91799 TEST=NetInternalsTest.NetInternalsPrerenderView (Currently flaky) Review URL: http://codereview.chromium.org/7550031 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@95721 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/browser/prerender/prerender_manager.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'chrome/browser/prerender/prerender_manager.cc') diff --git a/chrome/browser/prerender/prerender_manager.cc b/chrome/browser/prerender/prerender_manager.cc index dd19f27..cb83264 100644 --- a/chrome/browser/prerender/prerender_manager.cc +++ b/chrome/browser/prerender/prerender_manager.cc @@ -603,6 +603,7 @@ void PrerenderManager::MoveEntryToPendingDelete(PrerenderContents* entry) { break; } } + AddToHistory(entry); pending_delete_list_.push_back(entry); // Destroy the old TabContents relatively promptly to reduce resource usage, @@ -651,7 +652,6 @@ void PrerenderManager::DeletePendingDeleteEntries() { while (!pending_delete_list_.empty()) { PrerenderContents* contents = pending_delete_list_.front(); pending_delete_list_.pop_front(); - AddToHistory(contents); delete contents; } } -- cgit v1.1