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

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

Issue 2609713002: [Offline pages] Completely remove offliner factory. (Closed)
Patch Set: rebase update 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/test_request_coordinator_builder.cc
diff --git a/chrome/browser/android/offline_pages/test_request_coordinator_builder.cc b/chrome/browser/android/offline_pages/test_request_coordinator_builder.cc
index 58e45bab28cfd2dc4fe8cc93a912b052754159f5..2721a38c399e3d7f5ac365f8760c8836d7e7f9ec 100644
--- a/chrome/browser/android/offline_pages/test_request_coordinator_builder.cc
+++ b/chrome/browser/android/offline_pages/test_request_coordinator_builder.cc
@@ -7,8 +7,8 @@
#include <utility>
#include "components/offline_pages/core/background/network_quality_provider_stub.h"
-#include "components/offline_pages/core/background/offliner_factory_stub.h"
#include "components/offline_pages/core/background/offliner_policy.h"
+#include "components/offline_pages/core/background/offliner_stub.h"
#include "components/offline_pages/core/background/request_coordinator.h"
#include "components/offline_pages/core/background/request_queue.h"
#include "components/offline_pages/core/background/request_queue_in_memory_store.h"
@@ -29,7 +29,7 @@ std::unique_ptr<KeyedService> BuildTestRequestCoordinator(
std::unique_ptr<RequestQueue> queue(new RequestQueue(std::move(store)));
// Initialize the rest with stubs.
- std::unique_ptr<OfflinerFactory> offliner_factory(new OfflinerFactoryStub());
+ std::unique_ptr<Offliner> offliner(new OfflinerStub());
std::unique_ptr<Scheduler> scheduler_stub(new SchedulerStub());
// NetworkQualityProviderStub should be set by the test on the context first.
@@ -37,7 +37,7 @@ std::unique_ptr<KeyedService> BuildTestRequestCoordinator(
NetworkQualityProviderStub::GetUserData(context);
return std::unique_ptr<RequestCoordinator>(new RequestCoordinator(
- std::move(policy), std::move(offliner_factory), std::move(queue),
+ std::move(policy), std::move(offliner), std::move(queue),
std::move(scheduler_stub), network_quality_provider));
}
« no previous file with comments | « chrome/browser/android/offline_pages/request_coordinator_factory.cc ('k') | components/offline_pages/core/background/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698