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

Unified Diff: content/browser/download/download_stats.h

Issue 2769933003: Add more UMA to record whether parallel download is completed/interrupted/cancelled (Closed)
Patch Set: rebase Created 3 years, 9 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
« no previous file with comments | « content/browser/download/download_item_impl.cc ('k') | content/browser/download/download_stats.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/download/download_stats.h
diff --git a/content/browser/download/download_stats.h b/content/browser/download/download_stats.h
index d2060aa7e3b33a11b5c21c55a6dc964fd3275970..8d3fea375ff41f7d821453763e67afe79a2ac1e8 100644
--- a/content/browser/download/download_stats.h
+++ b/content/browser/download/download_stats.h
@@ -91,7 +91,7 @@ enum DownloadCountTypes {
// candidates for partial resumption.
STRONG_VALIDATOR_AND_ACCEPTS_RANGES,
- // Count of downloads that uses parallel download requests.
+ // (Deprecated) Count of downloads that uses parallel download requests.
USES_PARALLEL_REQUESTS,
// Count of new downloads.
@@ -158,7 +158,8 @@ void RecordDownloadCompleted(const base::TimeTicks& start,
// Record INTERRUPTED_COUNT, |reason|, |received| and |total| bytes.
void RecordDownloadInterrupted(DownloadInterruptReason reason,
int64_t received,
- int64_t total);
+ int64_t total,
+ bool uses_parallel_requests);
// Record that a download has been classified as malicious.
void RecordMaliciousDownloadClassified(DownloadDangerType danger_type);
@@ -217,6 +218,9 @@ void RecordFileBandwidth(size_t length,
base::TimeDelta disk_write_time,
base::TimeDelta elapsed_time);
+// Increment one of the count for parallel download.
+void RecordParallelDownloadCount(DownloadCountTypes type);
+
// Records the bandwidth for parallel download and estimates the saved time at
// the file end. Does not count in any hash computation or file open/close time.
void RecordParallelDownloadStats(
« no previous file with comments | « content/browser/download/download_item_impl.cc ('k') | content/browser/download/download_stats.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698