Content Shell: Introduce LayoutTestDownloadManagerDelegate.

content/shell/browser/layout_test/layout_test_browser_context.cc

Index: content/shell/browser/layout_test/layout_test_browser_context.cc
diff --git a/content/shell/browser/layout_test/layout_test_browser_context.cc b/content/shell/browser/layout_test/layout_test_browser_context.cc
index 0fa9effbd57ff21e50665acac5a797401519b96b..fff7d40488fe03527dddf6771804d1237feb4d8a 100644
--- a/content/shell/browser/layout_test/layout_test_browser_context.cc
+++ b/content/shell/browser/layout_test/layout_test_browser_context.cc
@@ -10,7 +10,7 @@
 #include "base/logging.h"
 #include "base/path_service.h"
 #include "content/public/browser/resource_context.h"
-#include "content/shell/browser/shell_download_manager_delegate.h"
+#include "content/shell/browser/layout_test/layout_test_download_manager_delegate.h"
 #include "content/shell/browser/shell_url_request_context_getter.h"
 
 #if defined(OS_WIN)
@@ -35,11 +35,9 @@ LayoutTestBrowserContext::~LayoutTestBrowserContext() {
 DownloadManagerDelegate*
 LayoutTestBrowserContext::GetDownloadManagerDelegate() {
   if (!download_manager_delegate_.get()) {
-    download_manager_delegate_.reset(new ShellDownloadManagerDelegate());
+    download_manager_delegate_.reset(new LayoutTestDownloadManagerDelegate());
     download_manager_delegate_->SetDownloadManager(
         BrowserContext::GetDownloadManager(this));
-    // TODO(mkwst): We can avoid this bit in the future by defining a
-    // LayoutTestDownloadManagerDelegate.

[Mike West, 2014/10/10 11:11:47]

We couldn't avoid this bit; I didn't notice that the patch came from the browser
context object. :/

     download_manager_delegate_->SetDownloadBehaviorForTesting(
         GetPath().Append(FILE_PATH_LITERAL("downloads")));
   }