Add ServiceConfig to the DownloadService

Desktop / Chromium - David Trainor [chromium.org] - 15 June 2017 02:05 EDT

Expose some potentially relevant configuration information through the public API on DownloadService

Bug: Change-Id: Ic6fe1b631c23e9c90423a24afc5f0ca00bfa1021 Reviewed-on: https://chromium-review.googlesource.com/535895 Commit-Queue: David Trainor

ad6999b Add ServiceConfig to the DownloadService
components/download/components_unittests.filter | 3 +-
.../content/factory/download_service_factory.cc | 6 ++--
components/download/internal/BUILD.gn | 3 ++
components/download/internal/controller_impl.cc | 6 ++--
components/download/internal/controller_impl.h | 10 +++---
.../download/internal/controller_impl_unittest.cc | 7 ++--
.../download/internal/download_service_impl.cc | 11 +++++--
.../download/internal/download_service_impl.h | 12 +++++--
.../internal/download_service_impl_unittest.cc | 4 ++-
.../download/internal/service_config_impl.cc | 24 ++++++++++++++
components/download/internal/service_config_impl.h | 32 +++++++++++++++++++
.../internal/service_config_impl_unittest.cc | 23 ++++++++++++++
components/download/public/BUILD.gn | 1 +
components/download/public/download_service.h | 4 +++
components/download/public/service_config.h | 37 ++++++++++++++++++++++
15 files changed, 162 insertions(+), 21 deletions(-)

Upstream: git.chromium.org


  • Share