[downloads] Correctly plumb client ID for download file scanning.

chrome/browser/download/download_browsertest.cc

 // Copyright (c) 2012 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 "chrome/browser/download/download_browsertest.h"
 
 #include <stdint.h>
 #include <sstream>
 #include <utility>
 
@@ skipping 1161 matching lines @@
 
   // Check state.
   EXPECT_EQ(1, browser()->tab_strip_model()->count());
   base::FilePath file(FILE_PATH_LITERAL("download-test1.lib"));
   CheckDownload(browser(), file, file);
 }
 
 #if defined(OS_WIN)
 // Download a file and confirm that the zone identifier (on windows)
 // is set to internet.
-IN_PROC_BROWSER_TEST_F(DownloadTest, CheckInternetZone) {
+IN_PROC_BROWSER_TEST_F(DownloadTest, CheckInternetZone_DependsOnLocalConfig) {
   GURL url(URLRequestMockHTTPJob::GetMockUrl(kDownloadTest1Path));
 
   // Download the file and wait.  We do not expect the Select File dialog.
   DownloadAndWait(browser(), url);
 
   // Check state.  Special file state must be checked before CheckDownload,
   // as CheckDownload will delete the output file.
   EXPECT_EQ(1, browser()->tab_strip_model()->count());
   base::FilePath file(FILE_PATH_LITERAL("download-test1.lib"));
   base::FilePath downloaded_file(DestinationFile(browser(), file));
   if (base::VolumeSupportsADS(downloaded_file))
     EXPECT_TRUE(base::HasInternetZoneIdentifier(downloaded_file));
   CheckDownload(browser(), file, file);
 }
+
+// Downloading a file from the local host shouldn't cause the application of a
+// zone identifier.
+IN_PROC_BROWSER_TEST_F(DownloadTest, CheckLocalhostZone_DependsOnLocalConfig) {
+  embedded_test_server()->ServeFilesFromDirectory(GetTestDataDirectory());
+  ASSERT_TRUE(embedded_test_server()->Start());
+
+  // Assumes that localhost maps to 127.0.0.1. Otherwise the test will fail
+  // since EmbeddedTestServer is listening on that address.
+  GURL url =
+      embedded_test_server()->GetURL("localhost", "/downloads/a_zip_file.zip");
+  DownloadAndWait(browser(), url);
+  base::FilePath file(FILE_PATH_LITERAL("a_zip_file.zip"));
+  base::FilePath downloaded_file(DestinationFile(browser(), file));
+  EXPECT_FALSE(base::HasInternetZoneIdentifier(downloaded_file));
+}
+
+// Same as the test above, but uses a file:// URL to a local file.
+IN_PROC_BROWSER_TEST_F(DownloadTest, CheckLocalFileZone_DependsOnLocalConfig) {
+  base::FilePath source_file = GetTestDataDirectory()
+                                   .AppendASCII("downloads")
+                                   .AppendASCII("a_zip_file.zip");
+
+  GURL url = net::FilePathToFileURL(source_file);
+  DownloadAndWait(browser(), url);
+  base::FilePath file(FILE_PATH_LITERAL("a_zip_file.zip"));
+  base::FilePath downloaded_file(DestinationFile(browser(), file));
+  EXPECT_FALSE(base::HasInternetZoneIdentifier(downloaded_file));
+}
 #endif
 
 // Put up a Select File dialog when the file is downloaded, due to
 // downloads preferences settings.
 IN_PROC_BROWSER_TEST_F(DownloadTest, DownloadMimeTypeSelect) {
   // Re-enable prompting.
   browser()->profile()->GetPrefs()->SetBoolean(
       prefs::kPromptForDownload, true);
   GURL url(URLRequestMockHTTPJob::GetMockUrl(kDownloadTest1Path));
 
@@ skipping 2431 matching lines @@
           browser(), 1,
           content::DownloadTestObserver::ON_DANGEROUS_DOWNLOAD_DENY));
   ui_test_utils::NavigateToURL(browser(), extension_url);
 
   observer->WaitForFinished();
 
   // Download shelf should close.
   EXPECT_FALSE(browser()->window()->IsDownloadShelfVisible());
 }
 #endif  // defined(OS_CHROMEOS)