[Downloads] Do not store error responses during resumption.

content/browser/download/url_downloader.h

Index: content/browser/download/url_downloader.h
diff --git a/content/browser/download/url_downloader.h b/content/browser/download/url_downloader.h
index 787feb4c91b35918dbe2ee35f3f089994f2553be..5beaa5895d30ee173828269e460f0decfefa89c7 100644
--- a/content/browser/download/url_downloader.h
+++ b/content/browser/download/url_downloader.h
@@ -17,6 +17,8 @@
 #include "net/url_request/url_request.h"
 
 namespace content {
+class ByteStreamReader;
+struct DownloadCreateInfo;
 class DownloadManagerImpl;
 
 class UrlDownloader : public net::URLRequest::Delegate {
@@ -60,6 +62,11 @@ class UrlDownloader : public net::URLRequest::Delegate {
   void ResumeRequest();
   void CancelRequest();
 
+  // Called when the UrlDownloader is done with the request. Posts a task to
+  // remove itself from its download manager, which in turn would cause the
+  // UrlDownloader to be freed.
+  void Done();

[svaldez, 2016/01/13 17:29:18]

Can we call it Destroy since this doesn't necessarily mean the request was
completed.

[asanka, 2016/01/28 02:24:17]

Done.

+
   scoped_ptr<net::URLRequest> request_;
   base::WeakPtr<DownloadManagerImpl> manager_;
   uint32_t download_id_;