Parallel Download finch config parameters on Chrome client.

content/browser/download/parallel_download_job.cc

Index: content/browser/download/parallel_download_job.cc
diff --git a/content/browser/download/parallel_download_job.cc b/content/browser/download/parallel_download_job.cc
index 41ba75beceffe372b04a1a4a06f4c0cc739e4bd8..7383f3e513773626f2db2a7fc5369d06f681a21b 100644
--- a/content/browser/download/parallel_download_job.cc
+++ b/content/browser/download/parallel_download_job.cc
@@ -5,25 +5,17 @@
 #include "content/browser/download/parallel_download_job.h"
 
 #include "base/memory/ptr_util.h"
+#include "content/browser/download/parallel_download_utils.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/storage_partition.h"
 
 namespace content {
 
-namespace {
-
-// TODO(xingliu): Use finch parameters to configure constants.
-// Default number of requests in a parallel download, including the original
-// request.
-const int kParallelRequestCount = 2;
-
-}  // namespace
-
 ParallelDownloadJob::ParallelDownloadJob(
     DownloadItemImpl* download_item,
     std::unique_ptr<DownloadRequestHandleInterface> request_handle)
     : DownloadJobImpl(download_item, std::move(request_handle)),
-      request_num_(kParallelRequestCount) {}
+      request_num_(GetParallelRequestCountConfig()) {}

[qinmin, 2017/03/07 05:56:50]

nit: this variable is no longer needed, just call
GetParallelRequestCountConfig() in the code should be fine

[xingliu, 2017/03/07 18:44:08]

Done, makes sense.

 
 ParallelDownloadJob::~ParallelDownloadJob() = default;