Add a delay to send the parallel requests.

content/browser/download/parallel_download_utils.cc

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/download/parallel_download_utils.h"
 
 #include "base/metrics/field_trial_params.h"
 #include "base/strings/string_number_conversions.h"
+#include "base/time/time.h"
 #include "content/public/browser/download_save_info.h"
 #include "content/public/common/content_features.h"
 
 namespace content {
 
 namespace {
 
 // Finch parameter key value for minimum slice size in bytes to use parallel
 // download.
 const char kMinSliceSizeFinchKey[] = "min_slice_size";
 
 // Default value for |kMinSliceSizeFinchKey|, when no parameter is specified.
 const int64_t kMinSliceSizeParallelDownload = 2097152;
 
 // Finch parameter key value for number of parallel requests in a parallel
 // download, including the original request.
 const char kParallelRequestCountFinchKey[] = "request_count";
 
 // Default value for |kParallelRequestCountFinchKey|, when no parameter is
 // specified.
 const int kParallelRequestCount = 2;
 
+// Finch parameter key value for the delay time in milliseconds to send
+// parallel requests after response of the original request is handled.
+const char kParallelRequestDelayFinchKey[] = "parallel_request_delay";
+
+// Default value for parallel request delay.
+const base::TimeDelta kParallelRequestDelay =

[qinmin, 2017/03/15 05:49:18]

move this inside GetParallelRequestDelayConfig(),

[xingliu, 2017/03/15 17:48:52]

Done.

+    base::TimeDelta::FromMilliseconds(0);
+
 // TODO(qinmin): replace this with a comparator operator in
 // DownloadItem::ReceivedSlice.
 bool compareReceivedSlices(const DownloadItem::ReceivedSlice& lhs,
                            const DownloadItem::ReceivedSlice& rhs) {
   return lhs.offset < rhs.offset;
 }
 
 }  // namespace
 
 std::vector<DownloadItem::ReceivedSlice> FindSlicesToDownload(
@@ skipping 56 matching lines @@
 
 int GetParallelRequestCountConfig() {
   std::string finch_value = base::GetFieldTrialParamValueByFeature(
       features::kParallelDownloading, kParallelRequestCountFinchKey);
   int result;
   return !finch_value.empty() && base::StringToInt(finch_value, &result)
              ? result
              : kParallelRequestCount;
 }
 
+base::TimeDelta GetParallelRequestDelayConfig() {
+  std::string finch_value = base::GetFieldTrialParamValueByFeature(
+      features::kParallelDownloading, kParallelRequestDelayFinchKey);
+  int64_t time_ms = 0;
+  return !finch_value.empty() && base::StringToInt64(finch_value, &time_ms)

[David Trainor- moved to gerrit, 2017/03/15 03:47:38]

base::StringToInt64 will return false if you pass an empty string.

[xingliu, 2017/03/15 17:48:52]

Done, oh, i see, removed !finch_value.empty().

+             ? base::TimeDelta::FromMilliseconds(time_ms)
+             : kParallelRequestDelay;
+}
+
 }  // namespace content