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

Unified Diff: components/download/internal/download_service_impl.h

Issue 2895953004: Add initial Controller to DownloadService (Closed)
Patch Set: Moved stats out Created 3 years, 7 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: components/download/internal/download_service_impl.h
diff --git a/components/download/internal/download_service_impl.h b/components/download/internal/download_service_impl.h
index 319aea1976b78b2692419696e30937540185bc85..bcee4eb5625c2aa6a049bb211d428fd00d664a0d 100644
--- a/components/download/internal/download_service_impl.h
+++ b/components/download/internal/download_service_impl.h
@@ -9,21 +9,23 @@
#include <string>
#include "base/macros.h"
-#include "components/download/internal/config.h"
#include "components/download/public/download_service.h"
namespace download {
+class Controller;
+
struct DownloadParams;
struct SchedulingParams;
// The internal implementation of the DownloadService.
class DownloadServiceImpl : public DownloadService {
public:
- DownloadServiceImpl(std::unique_ptr<Configuration> config);
+ DownloadServiceImpl(std::unique_ptr<Controller> scheduler);
xingliu 2017/05/27 00:25:32 nit: (std::unique_ptr<Controller> controller);
David Trainor- moved to gerrit 2017/05/30 18:55:23 Done.
~DownloadServiceImpl() override;
// DownloadService implementation.
+ ServiceStatus GetStatus() override;
void StartDownload(const DownloadParams& download_params) override;
void PauseDownload(const std::string& guid) override;
void ResumeDownload(const std::string& guid) override;
@@ -32,7 +34,7 @@ class DownloadServiceImpl : public DownloadService {
const SchedulingParams& params) override;
private:
- std::unique_ptr<Configuration> config_;
+ std::unique_ptr<Controller> controller_;
DISALLOW_COPY_AND_ASSIGN(DownloadServiceImpl);
};

Powered by Google App Engine
This is Rietveld 408576698