Issue 793158 link

Starred by 1 user

Issue metadata

Status:	Fixed
Owner:	xingliu@chromium.org
Closed:	Jan 2018
Cc:	shaktisahu@chromium.org dtrainor@chromium.org carlosk@chromium.org
Components:	UI>Browser>Downloads
EstimatedDays:	----
NextAction:	----
OS:	Android
Pri:	3
Type:	Bug

Expose more configuration in download service.

Project Member Reported by xingliu@chromium.org, Dec 8 2017

Issue description

Expose more configuration for download service.

Prefetch team requests to expose max_concurrent_downloads and max_running_downloads configuration in download service.

Currently download::ServiceConfig has already expose two config, we can simply add these two in there.

Project Member

Comment 1 by bugdroid1@chromium.org, Dec 11 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b531336e62fa18100c963e5eee2327dad614aa09

commit b531336e62fa18100c963e5eee2327dad614aa09
Author: Xing Liu <xingliu@chromium.org>
Date: Mon Dec 11 23:34:17 2017

Download service: Expose another configuration to client.



Bug:  793158 
Change-Id: I32c0567bb5e1c44ccd21e898e1022593121b187e
Reviewed-on: https://chromium-review.googlesource.com/817359
Commit-Queue: Xing Liu <xingliu@chromium.org>
Reviewed-by: David Trainor <dtrainor@chromium.org>
Cr-Commit-Position: refs/heads/master@{#523250}
[modify] https://crrev.com/b531336e62fa18100c963e5eee2327dad614aa09/components/download/internal/service_config_impl.cc
[modify] https://crrev.com/b531336e62fa18100c963e5eee2327dad614aa09/components/download/internal/service_config_impl.h
[modify] https://crrev.com/b531336e62fa18100c963e5eee2327dad614aa09/components/download/internal/service_config_impl_unittest.cc
[modify] https://crrev.com/b531336e62fa18100c963e5eee2327dad614aa09/components/download/public/service_config.h
[modify] https://crrev.com/b531336e62fa18100c963e5eee2327dad614aa09/components/download/public/test/test_download_service.cc

Comment 2 by xingliu@chromium.org, Jan 22 2018

Status: Fixed (was: Assigned)

►

Issue 793158 link

Issue metadata

Expose more configuration in download service.

Issue description

Comment 1 by bugdroid1@chromium.org, Dec 11 2017

Comment 1 Deleted

Comment 2 by xingliu@chromium.org, Jan 22 2018

Comment 2 Deleted

Sign in to add a comment