Add URLFetcher::SaveResponseToFileAtPath

content/public/common/url_fetcher.h

Index: content/public/common/url_fetcher.h
diff --git a/content/public/common/url_fetcher.h b/content/public/common/url_fetcher.h
index 571a34bc85624b09e67fca77634229174a9a999f..01bd81d316756c56b76312f392712dca0d98fe3d 100644
--- a/content/public/common/url_fetcher.h
+++ b/content/public/common/url_fetcher.h
@@ -169,8 +169,20 @@ class CONTENT_EXPORT URLFetcher {
   virtual base::TimeDelta GetBackoffDelay() const = 0;
 
   // By default, the response is saved in a string. Call this method to save the
+  // response to a file instead. Must be called before Start().
+  // |file_message_loop_proxy| will be used for all file operations.
+  // To save to a temporary file, use SaveResponseToTemporaryFile().
+  // The created file is removed when the URLFetcher is deleted unless you
+  // take the ownership.

[Sam Kerner (Chrome), 2012/03/06 19:44:31]

Please say how you take ownership:  "take ownership by calling
GetResponseAsFilePath()."

[hashimoto, 2012/03/07 04:39:21]

Done.

+  virtual void SaveResponseToFileAtPath(
+      const FilePath& file_path,
+      scoped_refptr<base::MessageLoopProxy> file_message_loop_proxy) = 0;
+
+  // By default, the response is saved in a string. Call this method to save the
   // response to a temporary file instead. Must be called before Start().
   // |file_message_loop_proxy| will be used for all file operations.
+  // The created file is removed when the URLFetcher is deleted unless you
+  // take the ownership.
   virtual void SaveResponseToTemporaryFile(
       scoped_refptr<base::MessageLoopProxy> file_message_loop_proxy) = 0;
 
@@ -227,7 +239,7 @@ class CONTENT_EXPORT URLFetcher {
 
   // Get the path to the file containing the response body. Returns false
   // if the response body was not saved to a file. If take_ownership is
-  // true, caller takes responsibility for the temp file, and it will not
+  // true, caller takes responsibility for the file, and it will not
   // be removed once the URLFetcher is destroyed.  User should not take
   // ownership more than once, or call this method after taking ownership.
   virtual bool GetResponseAsFilePath(bool take_ownership,