Basic tests for DownloadUrl/DownloadUrlToFile.

chrome/browser/download/download_browsertest.cc

 // Copyright (c) 2011 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 "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/file_path.h"
 #include "base/file_util.h"
 #include "base/memory/ref_counted.h"
 #include "base/path_service.h"
@@ skipping 449 matching lines @@
   // Returning false indicates a failure of the function, and should be asserted
   // in the caller.
   bool CheckDownload(Browser* browser,
                      const FilePath& downloaded_filename,
                      const FilePath& origin_filename) {
     // Find the path to which the data will be downloaded.
     FilePath downloaded_file(DestinationFile(browser, downloaded_filename));
 
     // Find the origin path (from which the data comes).
     FilePath origin_file(OriginFile(origin_filename));
+    return CheckDownloadFullPaths(browser, downloaded_file, origin_file);
+  }
+
+  // A version of CheckDownload that allows complete path specification.
+  bool CheckDownloadFullPaths(Browser* browser,
+                              const FilePath& downloaded_file,
+                              const FilePath& origin_file) {
     bool origin_file_exists = file_util::PathExists(origin_file);
     EXPECT_TRUE(origin_file_exists);
     if (!origin_file_exists)
       return false;
 
     // Confirm the downloaded data file exists.
     bool downloaded_file_exists = file_util::PathExists(downloaded_file);
     EXPECT_TRUE(downloaded_file_exists);
     if (!downloaded_file_exists)
       return false;
@@ skipping 1059 matching lines @@
   EXPECT_EQ(1u, observer->NumDangerousDownloadsSeen());
 
   // Download shelf should close. Download panel stays open on ChromeOS.
   CheckDownloadUI(browser(), false, true, FilePath());
 
   // Check that the extension was installed.
   ExtensionService* extension_service =
       browser()->profile()->GetExtensionService();
   ASSERT_TRUE(extension_service->GetExtensionById(kLargeThemeCrxId, false));
 }
+
+// Tests for download initiation functions.
+IN_PROC_BROWSER_TEST_F(DownloadTest, DownloadUrl) {
+  ASSERT_TRUE(InitialSetup(false));
+  FilePath file(FILE_PATH_LITERAL("download-test1.lib"));
+  GURL url(URLRequestMockHTTPJob::GetMockUrl(file));
+
+  // DownloadUrl always prompts; return acceptance of whatever it prompts.
+  NullSelectFile(browser());
+
+  TabContents* tab_contents = browser()->GetSelectedTabContents();
+  ASSERT_TRUE(tab_contents);
+
+  DownloadTestObserver* observer(
+    new DownloadTestObserver(
+        DownloadManagerForBrowser(browser()), 1,
+        DownloadItem::COMPLETE,  // Really done
+        false,                   // Ignore select file.
+        DownloadTestObserver::ON_DANGEROUS_DOWNLOAD_FAIL));
+  DownloadManagerForBrowser(browser())->DownloadUrl(
+      url, GURL(""), "", tab_contents);

[achuithb, 2011/11/06 20:44:00]

nit: shouldn't GURL() suffice?

+  observer->WaitForFinished();
+  EXPECT_TRUE(observer->select_file_dialog_seen());
+
+  // Check state.
+  EXPECT_EQ(1, browser()->tab_count());
+  ASSERT_TRUE(CheckDownload(browser(), file, file));
+  CheckDownloadUI(browser(), true, true, file);
+}
+
+IN_PROC_BROWSER_TEST_F(DownloadTest, DownloadUrlToFile) {
+  ASSERT_TRUE(InitialSetup(false));
+  FilePath file(FILE_PATH_LITERAL("download-test1.lib"));
+  GURL url(URLRequestMockHTTPJob::GetMockUrl(file));
+
+  TabContents* tab_contents = browser()->GetSelectedTabContents();
+  ASSERT_TRUE(tab_contents);
+
+  ScopedTempDir other_directory;
+  ASSERT_TRUE(other_directory.CreateUniqueTempDir());
+  FilePath target_file_full_path
+      = other_directory.path().Append(file.BaseName());
+  DownloadSaveInfo save_info;
+  save_info.file_path = target_file_full_path;
+
+  DownloadTestObserver* observer(CreateWaiter(browser(), 1));
+  DownloadManagerForBrowser(browser())->DownloadUrlToFile(
+      url, GURL(""), "", save_info, tab_contents);

[achuithb, 2011/11/06 20:44:00]

nit: same

+  observer->WaitForFinished();
+
+  // Check state.
+  EXPECT_EQ(1, browser()->tab_count());
+  ASSERT_TRUE(CheckDownloadFullPaths(browser(),
+                                     target_file_full_path,
+                                     OriginFile(file)));
+  // Temporary downloads won't be visible.
+  CheckDownloadUI(browser(), false, false, file);
+}