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 1e4d25633666f742df462f8cb8cd67a28b347030..ff9134d2de145a1359186fa96ebcb0201b190b96 100644 |
| --- a/components/offline_pages/background/request_coordinator.cc |
| +++ b/components/offline_pages/background/request_coordinator.cc |
| @@ -5,16 +5,21 @@ |
| #include "components/offline_pages/background/request_coordinator.h" |
| // TODO(fgorski): include file with SavePageRequest definition. |
| +#include "components/offline_pages/background/offliner_policy.h" |
| +#include "components/offline_pages/background/scheduler.h" |
|
fgorski
2016/05/03 22:48:16
I don't see this being used.
Pete Williamson
2016/05/04 00:39:21
Done.
|
| namespace offline_pages { |
| -// TODO(dougarnett): How to inject Offliner factories and policy objects. |
| -RequestCoordinator::RequestCoordinator() { |
| +RequestCoordinator::RequestCoordinator( |
| + OfflinerPolicy* policy, OfflinerFactory* factory) { |
| // Do setup as needed. |
| + // TODO(petewil): Assert policy not null. |
| + policy_ = policy; |
| + factory_ = factory; |
| } |
| RequestCoordinator::~RequestCoordinator() { |
| - // Do cleanup as needed. |
| + delete policy_; |
| } |
| bool RequestCoordinator::SavePageLater(const SavePageRequest& request) { |