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

Unified Diff: chrome/browser/android/offline_pages/background_loader_offliner_unittest.cc

Issue 2624833002: Revert of [Offline Pages] Remove need for OfflinePolicy when constructing an Offliner. (Closed)
Patch Set: Created 3 years, 11 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/android/offline_pages/background_loader_offliner_unittest.cc
diff --git a/chrome/browser/android/offline_pages/background_loader_offliner_unittest.cc b/chrome/browser/android/offline_pages/background_loader_offliner_unittest.cc
index 80e17b1a1546203993b92c2b2f4ff1a949af9cd3..260f13de0eb13721cd021d60a39f9476957d50cd 100644
--- a/chrome/browser/android/offline_pages/background_loader_offliner_unittest.cc
+++ b/chrome/browser/android/offline_pages/background_loader_offliner_unittest.cc
@@ -78,6 +78,7 @@
public:
explicit TestBackgroundLoaderOffliner(
content::BrowserContext* browser_context,
+ const OfflinerPolicy* policy,
OfflinePageModel* offline_page_model);
~TestBackgroundLoaderOffliner() override;
content::WebContentsTester* web_contents() {
@@ -95,8 +96,9 @@
TestBackgroundLoaderOffliner::TestBackgroundLoaderOffliner(
content::BrowserContext* browser_context,
+ const OfflinerPolicy* policy,
OfflinePageModel* offline_page_model)
- : BackgroundLoaderOffliner(browser_context, offline_page_model) {}
+ : BackgroundLoaderOffliner(browser_context, policy, offline_page_model) {}
TestBackgroundLoaderOffliner::~TestBackgroundLoaderOffliner() {}
@@ -153,7 +155,7 @@
void BackgroundLoaderOfflinerTest::SetUp() {
model_ = new MockOfflinePageModel();
- offliner_.reset(new TestBackgroundLoaderOffliner(profile(), model_));
+ offliner_.reset(new TestBackgroundLoaderOffliner(profile(), nullptr, model_));
}
void BackgroundLoaderOfflinerTest::OnCompletion(

Powered by Google App Engine
This is Rietveld 408576698