[Downloads] Retry renames after transient failures.

content/browser/download/download_stats.h

Index: content/browser/download/download_stats.h
diff --git a/content/browser/download/download_stats.h b/content/browser/download/download_stats.h
index 65fad8d6b4a598c3b65c60b54cb039422d98629c..9faf69c6e5ed8386f7a3bbb3eb24c84ab99bd05f 100644
--- a/content/browser/download/download_stats.h
+++ b/content/browser/download/download_stats.h
@@ -200,6 +200,11 @@ void RecordFileBandwidth(size_t length,
                          base::TimeDelta disk_write_time,
                          base::TimeDelta elapsed_time);
 
+// Record the result of a download file rename.

[Randy Smith (Not in Mondays), 2014/06/10 19:22:02]

I think there should probably be some doc in this comment about the
conditionalization on time_of_first_failure null values and why.  (Or maybe
rethinking calling structure; it's a little funny to have a bail this low in the
callstack dependent on a null value in a time argument).

[asanka, 2014/06/10 22:36:21]

Yeah. I changed the call stack and the arguments.

+void RecordDownloadFileRenameResult(
+    base::TimeTicks time_of_first_failure,
+    DownloadInterruptReason interrupt_reason);
+
 enum SavePackageEvent {
   // The user has started to save a page as a package.
   SAVE_PACKAGE_STARTED,