Send safe browsing ThreatDetails to track download CTR when user tries to recover blocked downloads

chrome/browser/extensions/api/downloads/downloads_api_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.
 
 // Disable everything on windows only. http://crbug.com/306144
 #ifndef OS_WIN
 
 #include <algorithm>
 
 #include "base/files/file_util.h"
@@ skipping 4022 matching lines @@
   // TODO(benjhayden) Test incognito.
 }
 
 // TODO(benjhayden) Figure out why DisableExtension() does not fire
 // OnListenerRemoved.
 
 // TODO(benjhayden) Test that the shelf is shown for download() both with and
 // without a WebContents.
 
 void OnDangerPromptCreated(DownloadDangerPrompt* prompt) {
-  prompt->InvokeActionForTesting(DownloadDangerPrompt::ACCEPT);
+  GURL unused_url = GURL::EmptyGURL();

[msw, 2015/11/17 01:17:38]

nit: can you just inline the default ctor below? ie. GURL()

[Jialiu Lin, 2015/11/17 02:05:50]

Done.

+  prompt->InvokeActionForTesting(DownloadDangerPrompt::ACCEPT, unused_url);
 }
 
 #if defined(OS_MACOSX)
 // Flakily triggers and assert on Mac.
 // http://crbug.com/180759
 #define MAYBE_DownloadExtensionTest_AcceptDanger DISABLED_DownloadExtensionTest_AcceptDanger
 #else
 #define MAYBE_DownloadExtensionTest_AcceptDanger DownloadExtensionTest_AcceptDanger
 #endif
 IN_PROC_BROWSER_TEST_F(DownloadExtensionTest,
@@ skipping 124 matching lines @@
   EXPECT_EQ(downloads::FILENAME_CONFLICT_ACTION_PROMPT, conflict_action);
   EXPECT_FALSE(warnings.empty());
   EXPECT_EQ(Warning::kDownloadFilenameConflict,
             warnings.begin()->warning_type());
   EXPECT_EQ("incumbent", warnings.begin()->extension_id());
 }
 
 }  // namespace extensions
 
 #endif  // http://crbug.com/306144