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

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

Issue 2044613003: Adds unit tests for offline_pages::PrerenderAdapter calling PrerenderManager. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Renamed SetPrerenderContentsFactory() to *ForTest() and made it public (and dropped two friends) Created 4 years, 6 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_browsertest.cc
diff --git a/chrome/browser/prerender/prerender_browsertest.cc b/chrome/browser/prerender/prerender_browsertest.cc
index 5425aac562da8478f25d7308337e359ca7f878ab..9c331c148186f4b4c48e9db4058c367f48a495bc 100644
--- a/chrome/browser/prerender/prerender_browsertest.cc
+++ b/chrome/browser/prerender/prerender_browsertest.cc
@@ -1134,7 +1134,8 @@ class PrerenderBrowserTest : virtual public InProcessBrowserTest {
prerender_manager->mutable_config().rate_limit_enabled = false;
ASSERT_TRUE(prerender_contents_factory_ == NULL);
prerender_contents_factory_ = new TestPrerenderContentsFactory;
- prerender_manager->SetPrerenderContentsFactory(prerender_contents_factory_);
+ prerender_manager->SetPrerenderContentsFactoryForTest(
+ prerender_contents_factory_);
ASSERT_TRUE(safe_browsing_factory_->test_safe_browsing_service());
}

Powered by Google App Engine
This is Rietveld 408576698