OnDownloadStarted takes DownloadItem* instead of DownloadId

content/public/browser/download_url_parameters.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_PUBLIC_BROWSER_DOWNLOAD_URL_PARAMETERS_H_
 #define CONTENT_PUBLIC_BROWSER_DOWNLOAD_URL_PARAMETERS_H_
 
 #include <string>
 #include <vector>
 
 #include "base/basictypes.h"
 #include "base/callback.h"
-#include "content/public/browser/download_id.h"
 #include "content/public/browser/download_save_info.h"
 #include "content/public/common/referrer.h"
 #include "googleurl/src/gurl.h"
 #include "net/base/net_errors.h"
 
 namespace content {
 
+class DownloadItem;
 class ResourceContext;
 class ResourceDispatcherHost;
 class WebContents;
 
 // Pass an instance of DownloadUrlParameters to DownloadManager::DownloadUrl()
 // to download the content at |url|. All parameters with setters are optional.
 // |referrer| and |referrer_encoding| are the referrer for the download. If
 // |prefer_cache| is true, then if the response to |url| is in the HTTP cache it
 // will be used without revalidation. If |post_id| is non-negative, then it
 // identifies the post transaction used to originally retrieve the |url|
 // resource - it also requires |prefer_cache| to be |true| since re-post'ing is
 // not done.  |save_info| specifies where the downloaded file should be saved,
 // and whether the user should be prompted about the download.  If not null,
 // |callback| will be called when the download starts, or if an error occurs
 // that prevents a download item from being created.  We send a pointer to
 // content::ResourceContext instead of the usual reference so that a copy of the
 // object isn't made.
 
 class CONTENT_EXPORT DownloadUrlParameters {
  public:
-  // The DownloadId will be invalid if-and-only-if there is an error.
-  typedef base::Callback<void(DownloadId, net::Error)> OnStartedCallback;
+  // If there is an error, then |item| will be NULL.
+  typedef base::Callback<void(DownloadItem*, net::Error)> OnStartedCallback;
 
   typedef std::pair<std::string, std::string> RequestHeadersNameValuePair;
   typedef std::vector<RequestHeadersNameValuePair> RequestHeadersType;
 
   static DownloadUrlParameters* FromWebContents(
       WebContents* web_contents,
       const GURL& url,
       const DownloadSaveInfo& save_info);
 
   DownloadUrlParameters(
@@ skipping 75 matching lines @@
   ResourceDispatcherHost* resource_dispatcher_host_;
   DownloadSaveInfo save_info_;
   GURL url_;
 
   DISALLOW_COPY_AND_ASSIGN(DownloadUrlParameters);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_DOWNLOAD_URL_PARAMETERS_H_