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

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

Issue 2624523003: [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 260f13de0eb13721cd021d60a39f9476957d50cd..80e17b1a1546203993b92c2b2f4ff1a949af9cd3 100644
--- a/chrome/browser/android/offline_pages/background_loader_offliner_unittest.cc
+++ b/chrome/browser/android/offline_pages/background_loader_offliner_unittest.cc
@@ -78,7 +78,6 @@ class TestBackgroundLoaderOffliner : public BackgroundLoaderOffliner {
public:
explicit TestBackgroundLoaderOffliner(
content::BrowserContext* browser_context,
- const OfflinerPolicy* policy,
OfflinePageModel* offline_page_model);
~TestBackgroundLoaderOffliner() override;
content::WebContentsTester* web_contents() {
@@ -96,9 +95,8 @@ class TestBackgroundLoaderOffliner : public BackgroundLoaderOffliner {
TestBackgroundLoaderOffliner::TestBackgroundLoaderOffliner(
content::BrowserContext* browser_context,
- const OfflinerPolicy* policy,
OfflinePageModel* offline_page_model)
- : BackgroundLoaderOffliner(browser_context, policy, offline_page_model) {}
+ : BackgroundLoaderOffliner(browser_context, offline_page_model) {}
TestBackgroundLoaderOffliner::~TestBackgroundLoaderOffliner() {}
@@ -155,7 +153,7 @@ BackgroundLoaderOfflinerTest::~BackgroundLoaderOfflinerTest() {}
void BackgroundLoaderOfflinerTest::SetUp() {
model_ = new MockOfflinePageModel();
- offliner_.reset(new TestBackgroundLoaderOffliner(profile(), nullptr, model_));
+ offliner_.reset(new TestBackgroundLoaderOffliner(profile(), model_));
}
void BackgroundLoaderOfflinerTest::OnCompletion(

Powered by Google App Engine
This is Rietveld 408576698