Clean up unused members and constructors from DownloadCreateInfo and DownloadSaveInfo

content/browser/download/download_types.h

Index: content/browser/download/download_types.h
diff --git a/content/browser/download/download_types.h b/content/browser/download/download_types.h
index f8c3343accba025b294d4a9c51d7b06b75b6e391..f3f72cd85f244d595c3a10a278e9d15b84f00ef5 100644
--- a/content/browser/download/download_types.h
+++ b/content/browser/download/download_types.h
@@ -19,14 +19,18 @@ struct CONTENT_EXPORT DownloadSaveInfo {
   DownloadSaveInfo();
   ~DownloadSaveInfo();
 
-  // This is usually the tentative final name, but not during resumption
-  // where it will be the intermediate file name.
+  // If non-empty, contains the full target path of the download that has been
+  // determined prior to download initiation. This is considered to be a trusted
+  // path.
   FilePath file_path;
 
-  linked_ptr<net::FileStream> file_stream;
-
+  // If non-empty, contains an untrusted filename suggestion. This can't contain
+  // a path (only a filename), and is only effective if |file_path| is empty.

[Randy Smith (Not in Mondays), 2012/02/08 18:53:54]

Thank you for improving these comments.

   string16 suggested_name;
 
+  // If non-NULL, contains the source data stream for the file contents.
+  linked_ptr<net::FileStream> file_stream;
+
   // The file offset at which to start the download.  May be 0.
   int64 offset;