Chromium Code Reviews| Index: components/offline_pages/background/request_coordinator.h |
| diff --git a/components/offline_pages/background/request_coordinator.h b/components/offline_pages/background/request_coordinator.h |
| index 44d937409fb97010c523edc2dec773567af07c30..1a3d4e48868bc6f3174d6008e40b6c62e8c03477 100644 |
| --- a/components/offline_pages/background/request_coordinator.h |
| +++ b/components/offline_pages/background/request_coordinator.h |
| @@ -21,6 +21,7 @@ class OfflinerPolicy; |
| class OfflinerFactory; |
| class Offliner; |
| class SavePageRequest; |
| +class Scheduler; |
| // Coordinates queueing and processing save page later requests. |
| class RequestCoordinator : |
| @@ -32,7 +33,8 @@ class RequestCoordinator : |
| RequestCoordinator(std::unique_ptr<OfflinerPolicy> policy, |
| std::unique_ptr<OfflinerFactory> factory, |
| - std::unique_ptr<RequestQueue> queue); |
| + std::unique_ptr<RequestQueue> queue, |
| + std::unique_ptr<Scheduler> scheduler); |
| ~RequestCoordinator() override; |
| @@ -54,6 +56,9 @@ class RequestCoordinator : |
| // Returns the request queue used for requests. Coordinator keeps ownership. |
| RequestQueue* GetQueue() { return queue_.get(); } |
| + Scheduler* GetSchedulerForTesting() { return scheduler_.get(); } |
| + |
|
fgorski
2016/05/12 14:32:35
nit: empty line
Pete Williamson
2016/05/12 17:21:18
Done.
|
| + |
| private: |
| void AddRequestResultCallback(RequestQueue::AddRequestResult result, |
| const SavePageRequest& request); |
| @@ -63,6 +68,8 @@ class RequestCoordinator : |
| std::unique_ptr<OfflinerFactory> factory_; |
| // RequestQueue. Used to store incoming requests. Owned. |
| std::unique_ptr<RequestQueue> queue_; |
| + // Scheduler is owned by the RequestCoordiator. |
|
fgorski
2016/05/12 14:32:35
nit: document as above.
Pete Williamson
2016/05/12 17:21:18
Done.
|
| + std::unique_ptr<Scheduler> scheduler_; |
| DISALLOW_COPY_AND_ASSIGN(RequestCoordinator); |
| }; |