diff options
author | tburkard@chromium.org <tburkard@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-01-25 02:16:17 +0000 |
---|---|---|
committer | tburkard@chromium.org <tburkard@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-01-25 02:16:17 +0000 |
commit | c237afd74ee094d8ad67ea946c6fbd1b5dacf11d (patch) | |
tree | 287b0942a3cb5c2c5b20e75721636eff02a85b3b /chrome/browser/prerender/prerender_manager_unittest.cc | |
parent | cc3e428082f586dbc2334be0f1306d6ed413b615 (diff) | |
download | chromium_src-c237afd74ee094d8ad67ea946c6fbd1b5dacf11d.zip chromium_src-c237afd74ee094d8ad67ea946c6fbd1b5dacf11d.tar.gz chromium_src-c237afd74ee094d8ad67ea946c6fbd1b5dacf11d.tar.bz2 |
Testing these conditions must be done in the browser test.
I will add appropriate tests for this tomorrow.
Meanwhile, I am backing out these incorrect tests.
TBR=cbentzel@chromium.org, mmenke@chromium.org, dominich@chromium.org
Review URL: https://chromiumcodereview.appspot.com/9234014
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@118982 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/prerender/prerender_manager_unittest.cc')
-rw-r--r-- | chrome/browser/prerender/prerender_manager_unittest.cc | 32 |
1 files changed, 0 insertions, 32 deletions
diff --git a/chrome/browser/prerender/prerender_manager_unittest.cc b/chrome/browser/prerender/prerender_manager_unittest.cc index 5a9a908..bdedb0f 100644 --- a/chrome/browser/prerender/prerender_manager_unittest.cc +++ b/chrome/browser/prerender/prerender_manager_unittest.cc @@ -460,38 +460,6 @@ TEST_F(PrerenderManagerTest, ControlGroup) { EXPECT_FALSE(prerender_contents->has_started()); } -// Tests that a PrerenderManager created for a browser session in the control -// group works as expected if the site gets visited. -TEST_F(PrerenderManagerTest, ControlGroupVisited) { - RestorePrerenderMode restore_prerender_mode; - PrerenderManager::SetMode( - PrerenderManager::PRERENDER_MODE_EXPERIMENT_CONTROL_GROUP); - GURL url("http://www.google.com/"); - DummyPrerenderContents* prerender_contents = - prerender_manager()->CreateNextPrerenderContents( - url, - FINAL_STATUS_WOULD_HAVE_BEEN_USED); - EXPECT_TRUE(prerender_manager()->AddSimplePrerender(url)); - EXPECT_FALSE(prerender_contents->has_started()); - ASSERT_EQ(prerender_contents, prerender_manager()->GetEntry(url)); -} - -// Tests that a PrerenderManager correctly handles the case of visitiung a page -// when the NoUseGroup is enabled. -TEST_F(PrerenderManagerTest, NoUseGroupVisited) { - RestorePrerenderMode restore_prerender_mode; - PrerenderManager::SetMode( - PrerenderManager::PRERENDER_MODE_EXPERIMENT_NO_USE_GROUP); - GURL url("http://www.google.com/"); - DummyPrerenderContents* prerender_contents = - prerender_manager()->CreateNextPrerenderContents( - url, - FINAL_STATUS_WOULD_HAVE_BEEN_USED); - EXPECT_TRUE(prerender_manager()->AddSimplePrerender(url)); - EXPECT_TRUE(prerender_contents->has_started()); - ASSERT_EQ(prerender_contents, prerender_manager()->GetEntry(url)); -} - // Tests that prerendering is cancelled when the source render view does not // exist. On failure, the DCHECK in CreatePrerenderContents() above should be // triggered. |