Remove the BeginSaveFile and BeginDownload methods from ResourceDispatcherHostImpl

content/browser/download/download_resource_handler.h

Index: content/browser/download/download_resource_handler.h
diff --git a/content/browser/download/download_resource_handler.h b/content/browser/download/download_resource_handler.h
index 92663589910fad3df31db8ef1821cacf37c7549e..084e4b3fd6d50648710daff42d23a3537b3e2ab3 100644
--- a/content/browser/download/download_resource_handler.h
+++ b/content/browser/download/download_resource_handler.h
@@ -42,6 +42,16 @@ class CONTENT_EXPORT DownloadResourceHandler
   // |id| should be invalid if the id should be automatically assigned.
   DownloadResourceHandler(net::URLRequest* request);
 
+  // static
+  // This function is passed into ResourceDispatcherHostImpl during its
+  // creation and is used to create instances of DownloadResourceHandler as
+  // needed.
+  // TODO(ananta)
+  // Find a better way to achieve this. Ideally we want to move the logic of
+  // creating DownloadResourceHandler instances out of
+  // ResourceDispatcherHostImpl.
+  static std::unique_ptr<ResourceHandler> Create(net::URLRequest* request);
+
   bool OnRequestRedirected(const net::RedirectInfo& redirect_info,
                            ResourceResponse* response,
                            bool* defer) override;