Chromium Code Reviews| Index: components/offline_pages/background/request_coordinator.cc |
| diff --git a/components/offline_pages/background/request_coordinator.cc b/components/offline_pages/background/request_coordinator.cc |
| index bf447575773bc00e0defe3b16d72bbbdf0695afa..ef6a5da5e357d94b0db0dc96b512c59bf63563e4 100644 |
| --- a/components/offline_pages/background/request_coordinator.cc |
| +++ b/components/offline_pages/background/request_coordinator.cc |
| @@ -10,18 +10,22 @@ |
| #include "components/offline_pages/background/offliner_factory.h" |
| #include "components/offline_pages/background/offliner_policy.h" |
| #include "components/offline_pages/background/save_page_request.h" |
| +#include "components/offline_pages/background/scheduler.h" |
| #include "components/offline_pages/offline_page_item.h" |
| namespace offline_pages { |
| -RequestCoordinator::RequestCoordinator(std::unique_ptr<OfflinerPolicy> policy, |
| - std::unique_ptr<OfflinerFactory> factory, |
| - std::unique_ptr<RequestQueue> queue) { |
| +RequestCoordinator::RequestCoordinator( |
| + std::unique_ptr<OfflinerPolicy> policy, |
| + std::unique_ptr<OfflinerFactory> factory, |
| + std::unique_ptr<RequestQueue> queue, |
| + std::unique_ptr<Scheduler> scheduler) { |
| // Do setup as needed. |
| // TODO(petewil): Assert policy not null. |
|
fgorski
2016/05/12 14:32:35
can you already do this?
Pete Williamson
2016/05/12 17:21:18
Done.
|
| policy_ = std::move(policy); |
| factory_ = std::move(factory); |
| queue_ = std::move(queue); |
| + scheduler_ = std::move(scheduler); |
|
fgorski
2016/05/12 14:32:35
C++: move all assignments to the initializer list.
Pete Williamson
2016/05/12 17:21:18
Done.
|
| } |
| RequestCoordinator::~RequestCoordinator() {} |
| @@ -44,6 +48,9 @@ bool RequestCoordinator::SavePageLater( |
| AsWeakPtr())); |
| // TODO: Do I need to persist the request in case the add fails? |
| + Scheduler::TriggerCondition conditions; |
| + scheduler_->Schedule(conditions); |
| + |
| return true; |
| } |