Put history insertion for downloads processing inline.

chrome/browser/download/download_manager_unittest.cc

Index: chrome/browser/download/download_manager_unittest.cc
diff --git a/chrome/browser/download/download_manager_unittest.cc b/chrome/browser/download/download_manager_unittest.cc
index 210788d02f2b805e9302f78b0276fe8357cecb4a..42ac9a7444860b457d5f1918b8f7ba251e64d67b 100644
--- a/chrome/browser/download/download_manager_unittest.cc
+++ b/chrome/browser/download/download_manager_unittest.cc
@@ -152,28 +152,29 @@ const struct {
   bool will_delete_crdownload;
   int expected_rename_count;
 } kDownloadRenameCases[] = {
-  // Safe download, download finishes BEFORE rename.
-  // Needs to be renamed only once.  Crdownload file needs to be deleted.
+  // Safe download, download finishes BEFORE file determined.

[Paweł Hajdan Jr., 2011/01/05 07:46:30]

nit: file determined -> file name determined?

[Randy Smith (Not in Mondays), 2011/01/13 02:13:24]

Done (all four cases).

+  // Renamed twice (linear path through UI).  Crdownload file does not need
+  // to be deleted.
   { FILE_PATH_LITERAL("foo.zip"),
     false,
     true,
-    true,
-    1, },
-  // Dangerous download, download finishes BEFORE rename.
+    false,
+    2, },
+  // Dangerous download, download finishes BEFORE file determined.
   // Needs to be renamed only once.
   { FILE_PATH_LITERAL("unconfirmed xxx.crdownload"),
     true,
     true,
     false,
     1, },
-  // Safe download, download finishes AFTER rename.
+  // Safe download, download finishes AFTER file determined.
   // Needs to be renamed twice.
   { FILE_PATH_LITERAL("foo.zip"),
     false,
     false,
     false,
     2, },
-  // Dangerous download, download finishes AFTER rename.
+  // Dangerous download, download finishes AFTER file determined.
   // Needs to be renamed only once.
   { FILE_PATH_LITERAL("unconfirmed xxx.crdownload"),
     true,