Add pending preloads indexed by routing id. Start preloading once we navigate.

chrome/browser/prerender/prerender_manager_unittest.cc

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/time.h"
 #include "chrome/browser/prerender/prerender_contents.h"
 #include "chrome/browser/prerender/prerender_manager.h"
 #include "content/browser/browser_thread.h"
+#include "content/browser/renderer_host/render_view_host.h"
+#include "content/browser/renderer_host/render_process_host.h"
 #include "googleurl/src/gurl.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace prerender {
 
 namespace {
 
 class DummyPrerenderContents : public PrerenderContents {
  public:
   DummyPrerenderContents(PrerenderManager* prerender_manager,
@@ skipping 15 matching lines @@
   }
 
   virtual ~DummyPrerenderContents() {
     EXPECT_EQ(expected_final_status_, final_status());
   }
 
   virtual void StartPrerendering() OVERRIDE {
     has_started_ = true;
   }
 
+  virtual bool child_id(int* child_id) const OVERRIDE {
+    *child_id = 0;
+    return true;
+  }
+
+  virtual bool route_id(int* route_id) const OVERRIDE {
+    *route_id = 0;
+    return true;
+  }
+
   bool has_started() const { return has_started_; }
 
  private:
   bool has_started_;
   FinalStatus expected_final_status_;
 };
 
 class TestPrerenderManager : public PrerenderManager {
  public:
   TestPrerenderManager()
       : PrerenderManager(NULL),
         time_(base::Time::Now()),
         next_pc_(NULL) {
   }
 
   void AdvanceTime(base::TimeDelta delta) {
     time_ += delta;
   }
 
   void SetNextPrerenderContents(PrerenderContents* pc) {
     next_pc_.reset(pc);
   }
 
   // Shorthand to add a simple preload with no aliases.
   bool AddSimplePreload(const GURL& url) {
     return AddPreload(url, std::vector<GURL>(), GURL());
   }
 
+  bool IsPendingEntry(const GURL& url) {
+    return (PrerenderManager::FindPendingEntry(url) != NULL);
+  }
+
   PrerenderContents* next_pc() { return next_pc_.get(); }
 
  protected:
   virtual ~TestPrerenderManager() {
     if (next_pc()) {
       next_pc()->set_final_status(
           FINAL_STATUS_MANAGER_SHUTDOWN);
     }
   }
 
@@ skipping 183 matching lines @@
   EXPECT_TRUE(prerender_manager_->AddPreload(url, alias_urls, GURL()));
   EXPECT_FALSE(prerender_manager_->AddSimplePreload(url));
   EXPECT_FALSE(prerender_manager_->AddSimplePreload(alias_url1));
   EXPECT_FALSE(prerender_manager_->AddSimplePreload(alias_url2));
   ASSERT_EQ(pc, prerender_manager_->GetEntry(url));
 
   pc->set_final_status(FINAL_STATUS_USED);
   delete pc;
 }
 
-}  // naemspace prerender
+TEST_F(PrerenderManagerTest, PendingPreloadTest) {
+  GURL url("http://www.google.com/");
+  DummyPrerenderContents* pc =
+      new DummyPrerenderContents(prerender_manager_.get(),
+                                 url,
+                                 FINAL_STATUS_USED);
+  prerender_manager_->SetNextPrerenderContents(pc);
+  EXPECT_TRUE(prerender_manager_->AddSimplePreload(url));
+
+  int child_id;
+  int route_id;
+  ASSERT_TRUE(pc->child_id(&child_id));
+  ASSERT_TRUE(pc->route_id(&route_id));
+
+  GURL pending_url("http://news.google.com/");
+
+  prerender_manager_->AddPendingPreload(std::make_pair(child_id, route_id),
+                                        pending_url,
+                                        std::vector<GURL>(),
+                                        url);
+
+  EXPECT_TRUE(prerender_manager_->IsPendingEntry(pending_url));
+  EXPECT_TRUE(pc->has_started());
+  ASSERT_EQ(pc, prerender_manager_->GetEntry(url));
+  pc->set_final_status(FINAL_STATUS_USED);
+
+  delete pc;
+}
+
+TEST_F(PrerenderManagerTest, PendingPreloadSkippedTest) {
+  GURL url("http://www.google.com/");
+  DummyPrerenderContents* pc =
+      new DummyPrerenderContents(prerender_manager_.get(),
+                                 url,
+                                 FINAL_STATUS_TIMED_OUT);
+  prerender_manager_->SetNextPrerenderContents(pc);
+
+  int child_id;
+  int route_id;
+  ASSERT_TRUE(pc->child_id(&child_id));
+  ASSERT_TRUE(pc->route_id(&route_id));
+
+  EXPECT_TRUE(prerender_manager_->AddSimplePreload(url));
+  prerender_manager_->AdvanceTime(prerender_manager_->max_prerender_age()
+                                  + base::TimeDelta::FromSeconds(1));
+  // GetEntry will cull old entries which should now include pc.
+  ASSERT_EQ(NULL, prerender_manager_->GetEntry(url));
+
+  GURL pending_url("http://news.google.com/");
+
+  prerender_manager_->AddPendingPreload(std::make_pair(child_id, route_id),
+                                        pending_url,
+                                        std::vector<GURL>(),
+                                        url);
+  EXPECT_FALSE(prerender_manager_->IsPendingEntry(pending_url));
+}
+
+}  // namespace prerender