Replace the DownloadFileManager with direct ownership of DownloadFileImpl

content/browser/download/download_item_impl_delegate.h

 // 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.
 
 #ifndef CONTENT_BROWSER_DOWNLOAD_DOWNLOAD_ITEM_IMPL_DELEGATE_H_
 #define CONTENT_BROWSER_DOWNLOAD_DOWNLOAD_ITEM_IMPL_DELEGATE_H_
 
 #include "base/callback.h"
 #include "base/file_path.h"
 #include "content/common/content_export.h"
+#include "content/public/browser/download_danger_type.h"
+#include "content/public/browser/download_item.h"
 
 class DownloadFileManager;
 class DownloadItemImpl;
 
 namespace content {
 class BrowserContext;
 }
 
 // Delegate for operations that a DownloadItemImpl can't do for itself.
 // The base implementation of this class does nothing (returning false
 // on predicates) so interfaces not of interest to a derived class may
 // be left unimplemented.
 class CONTENT_EXPORT DownloadItemImplDelegate {
  public:
+  typedef base::Callback<void(
+      const FilePath&,                  // Target path
+      content::DownloadItem::TargetDisposition,  // overwrite/uniquify target
+      content::DownloadDangerType,
+      const FilePath&                   // Intermediate file path
+                              )> DownloadTargetCallback;
+
   DownloadItemImplDelegate();
   virtual ~DownloadItemImplDelegate();
 
   // Used for catching use-after-free errors.
   void Attach();
   void Detach();
 
+  // Request determination of the download target from the delegate.
+  virtual void DetermineDownloadTarget(
+      DownloadItemImpl* download, const DownloadTargetCallback& callback);
+
   // Allows the delegate to delay completion of the download.  This function
   // will call the callback passed when the download is ready for completion.
   // This may be done immediately, from within the routine itself, or it
   // may be delayed.
   // This routine should only be called once per download.
   virtual void ReadyForDownloadCompletion(
       DownloadItemImpl* download,
       const base::Closure& complete_callback);
 
-  // Tests if a file type should be opened automatically.
-  virtual bool ShouldOpenFileBasedOnExtension(const FilePath& path);
-
   // Allows the delegate to override the opening of a download. If it returns
   // true then it's reponsible for opening the item.
   virtual bool ShouldOpenDownload(DownloadItemImpl* download);
 
+  // Tests if a file type should be opened automatically.
+  virtual bool ShouldOpenFileBasedOnExtension(const FilePath& path);
+
   // Checks whether a downloaded file still exists and updates the
   // file's state if the file is already removed.
   // The check may or may not result in a later asynchronous call
   // to OnDownloadedFileRemoved().
   virtual void CheckForFileRemoval(DownloadItemImpl* download_item);
 
   // For contextual issues like language and prefs.
   virtual content::BrowserContext* GetBrowserContext() const;
 
   // Get the DownloadFileManager to use for this download.
@@ skipping 16 matching lines @@
   virtual void AssertStateConsistent(DownloadItemImpl* download) const;
 
  private:
   // For "Outlives attached DownloadItemImpl" invariant assertion.
   int count_;
 
   DISALLOW_COPY_AND_ASSIGN(DownloadItemImplDelegate);
 };
 
 #endif  // CONTENT_BROWSER_DOWNLOAD_DOWNLOAD_ITEM_IMPL_DELEGATE_H_