Clean up unused members and constructors from DownloadCreateInfo and DownloadSaveInfo

content/browser/download/download_types.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_TYPES_H_
 #define CONTENT_BROWSER_DOWNLOAD_DOWNLOAD_TYPES_H_
 #pragma once
 
 #include "base/file_path.h"
 #include "base/memory/linked_ptr.h"
 #include "content/common/content_export.h"
 #include "net/base/file_stream.h"
 
 // Holds the information about how to save a download file.
 // In the case of download continuation, |file_path| is set to the current file
 // name, |offset| is set to the point where we left off, and |hash_state| will
 // hold the state of the hash algorithm where we left off.
 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;
 
+  // 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;
 
-  string16 suggested_name;
-
   // The file offset at which to start the download.  May be 0.
   int64 offset;
 
   // The state of the hash at the start of the download.  May be empty.
   std::string hash_state;
 
   // If |prompt_for_save_location| is true, and |file_path| is empty, then
   // the user will be prompted for a location to save the download. Otherwise,
   // the location will be determined automatically using |file_path| as a
   // basis if |file_path| is not empty.
   // |prompt_for_save_location| defaults to false.
   bool prompt_for_save_location;
 };
 
 #endif  // CONTENT_BROWSER_DOWNLOAD_DOWNLOAD_TYPES_H_