Add HTTP PUT request type to content:URLFetcher

content/public/common/url_fetcher.h

Index: content/public/common/url_fetcher.h
===================================================================
--- content/public/common/url_fetcher.h	(revision 120612)
+++ content/public/common/url_fetcher.h	(working copy)
@@ -76,6 +76,7 @@
     HEAD,
     DELETE_REQUEST,   // DELETE is already taken on Windows.
                       // <winnt.h> defines a DELETE macro.
+    PUT,
   };
 
   // |url| is the URL to send the request to.
@@ -142,6 +143,11 @@
   virtual void SetExtraRequestHeaders(
       const std::string& extra_request_headers) = 0;
 
+  // Add header (with format $key ":" $value) to the request headers.

[wtc, 2012/02/06 23:34:07]

Nit: I'd omit the dollar signs ($) in the format description.

You can consider using the exact wording from HTTP 1.1:

    field-name ":" [ field-value ]

[kuan, 2012/02/06 23:58:17]

Done.

+  // Must be called before the request is started.
+  virtual void AddRequestHeaderFromString(
+      const base::StringPiece& header_line) = 0;

[wtc, 2012/02/06 23:34:07]

It's important to explain the difference between this
method and the SetExtraRequestHeaders method above.
One important difference (ignoring the std::string vs.
base::StringPiece difference) is that SetExtraRequestHeaders
replaces the extra request headers whereas this method
appends one header.

I suggest renaming this method AddExtraRequestHeader.

I also suggest using the same data type (either std::string
or base::StringPiece) for both methods.

[kuan, 2012/02/06 23:58:17]

Done.

+
   virtual void GetExtraRequestHeaders(net::HttpRequestHeaders* headers) = 0;
 
   // Set the net::URLRequestContext on the request.  Must be called before the