diff options
author | cbentzel@chromium.org <cbentzel@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-06-30 18:09:11 +0000 |
---|---|---|
committer | cbentzel@chromium.org <cbentzel@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-06-30 18:09:11 +0000 |
commit | 940da33e066af01f31bf4c0f4d9ababf5e9feb6f (patch) | |
tree | 5b8bb29f70247ae4e6cb2aa0123c549436c28009 /chrome/browser/prerender/prerender_manager_unittest.cc | |
parent | df8996560ffbb5ed27b34f0722bf4af09de7905e (diff) | |
download | chromium_src-940da33e066af01f31bf4c0f4d9ababf5e9feb6f.zip chromium_src-940da33e066af01f31bf4c0f4d9ababf5e9feb6f.tar.gz chromium_src-940da33e066af01f31bf4c0f4d9ababf5e9feb6f.tar.bz2 |
Move some prerender configuration settings out of PrerenderManager.
BUG=None
TEST=Existing Tests.
Review URL: http://codereview.chromium.org/7248062
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91159 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 | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/chrome/browser/prerender/prerender_manager_unittest.cc b/chrome/browser/prerender/prerender_manager_unittest.cc index e275eb5..7da1493 100644 --- a/chrome/browser/prerender/prerender_manager_unittest.cc +++ b/chrome/browser/prerender/prerender_manager_unittest.cc @@ -66,7 +66,7 @@ class TestPrerenderManager : public PrerenderManager { time_ticks_(base::TimeTicks::Now()), next_prerender_contents_(NULL), prerender_tracker_(prerender_tracker) { - rate_limit_enabled_ = false; + set_rate_limit_enabled(false); } virtual ~TestPrerenderManager() { @@ -130,7 +130,9 @@ class TestPrerenderManager : public PrerenderManager { return (PrerenderManager::FindPendingEntry(url) != NULL); } - void set_rate_limit_enabled(bool enabled) { rate_limit_enabled_ = true; } + void set_rate_limit_enabled(bool enabled) { + mutable_config().rate_limit_enabled = enabled; + } PrerenderContents* next_prerender_contents() { return next_prerender_contents_.get(); @@ -257,8 +259,8 @@ TEST_F(PrerenderManagerTest, ExpireTest) { EXPECT_TRUE(prerender_manager()->AddSimplePrerender(url)); EXPECT_EQ(null, prerender_manager()->next_prerender_contents()); EXPECT_TRUE(prerender_contents->has_started()); - prerender_manager()->AdvanceTime(prerender_manager()->max_prerender_age() - + base::TimeDelta::FromSeconds(1)); + prerender_manager()->AdvanceTime(prerender_manager()->config().max_age + + base::TimeDelta::FromSeconds(1)); ASSERT_EQ(null, prerender_manager()->GetEntry(url)); } @@ -291,7 +293,7 @@ TEST_F(PrerenderManagerTest, DropOldestRequestTest) { // Two element prerender test. Ensure that the LRU operates correctly if we // permit 2 elements to be kept prerendered. TEST_F(PrerenderManagerTest, TwoElementPrerenderTest) { - prerender_manager()->set_max_elements(2); + prerender_manager()->mutable_config().max_elements = 2; GURL url("http://www.google.com/"); DummyPrerenderContents* prerender_contents = prerender_manager()->CreateNextPrerenderContents( |