[Downloads] Allow acquiring dangerous download file.

content/browser/download/download_browsertest.cc

Index: content/browser/download/download_browsertest.cc
diff --git a/content/browser/download/download_browsertest.cc b/content/browser/download/download_browsertest.cc
index 8f1e5427311429be20a03d2d998de5485dbe47e3..eec268d6b3a6b0044c5a2e9ef234b8752b7dd550 100644
--- a/content/browser/download/download_browsertest.cc
+++ b/content/browser/download/download_browsertest.cc
@@ -701,7 +701,7 @@ IN_PROC_BROWSER_TEST_F(DownloadContentTest, DownloadCancelled) {
   ASSERT_EQ(DownloadItem::IN_PROGRESS, downloads[0]->GetState());
 
   // Cancel the download and wait for download system quiesce.
-  downloads[0]->Delete(DownloadItem::DELETE_DUE_TO_USER_DISCARD);
+  downloads[0]->Cancel(true);

[Randy Smith (Not in Mondays), 2013/05/28 17:29:40]

Not a blocking comment, just for my curiosity: Why change this to Cancel()
instead of Remove() as you do elsewhere?  

[asanka, 2013/05/28 22:01:39]

I though that Cancel() is what agrees with the intention of the test. Both in
terms of the test name and comments, and also with the outcome being tested.

   scoped_refptr<DownloadTestFlushObserver> flush_observer(
       new DownloadTestFlushObserver(DownloadManagerForShell(shell())));
   flush_observer->WaitForFlush();
@@ -1542,7 +1542,7 @@ IN_PROC_BROWSER_TEST_F(DownloadContentTest, CancelResumingDownload) {
   // call, and that's for the second download created below.
   EXPECT_CALL(dm_observer, OnDownloadCreated(_,_)).Times(1);
   download->Resume();
-  download->Cancel(DownloadItem::DELETE_DUE_TO_USER_DISCARD);
+  download->Cancel(true);
 
   // The intermediate file should now be gone.
   RunAllPendingInMessageLoop(BrowserThread::FILE);