POSTs to HSTS domains are no longer converted to GETs

net/url_request/url_request_redirect_job.h

 // Copyright (c) 2011 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 NET_URL_REQUEST_URL_REQUEST_REDIRECT_JOB_H_
 #define NET_URL_REQUEST_URL_REQUEST_REDIRECT_JOB_H_
 
 #include "base/memory/weak_ptr.h"
 #include "net/base/net_export.h"
 #include "net/url_request/url_request_job.h"
 
 class GURL;
 
 namespace net {
 
 // A URLRequestJob that will redirect the request to the specified
 // URL.  This is useful to restart a request at a different URL based
 // on the result of another job.
 class NET_EXPORT URLRequestRedirectJob : public URLRequestJob {
  public:
   // Valid status codes for the redirect job. Other 30x codes are theoretically
-  // valid, but unused so far.
+  // valid, but unused so far.  Both 302 and 307 are temporary redirects, with
+  // the difference being that 302 converts POSTs to GETs and removes upload
+  // data.
   enum StatusCode {
     REDIRECT_302_FOUND = 302,
     REDIRECT_307_TEMPORARY_REDIRECT = 307,
   };
 
   // Constructs a job that redirects to the specified URL.
   URLRequestRedirectJob(URLRequest* request,
                         NetworkDelegate* network_delegate,
-                        const GURL& redirect_destination);
-
-  // Change the HTTP status code to use for the redirect. Default is
-  // REDIRECT_302_FOUND.
-  void set_redirect_code(StatusCode code) {
-    http_status_code_ = static_cast<int>(code);
-  }
+                        const GURL& redirect_destination,
+                        StatusCode http_status_code);
 
   virtual void Start() OVERRIDE;
   virtual bool IsRedirectResponse(GURL* location,
                                   int* http_status_code) OVERRIDE;
 
  private:
   virtual ~URLRequestRedirectJob();
 
   void StartAsync();
 
   GURL redirect_destination_;
   int http_status_code_;

[willchan no longer on Chromium, 2012/11/17 21:43:08]

Nit: Make these member variables const?
Up to you, but I like documenting that these can't change.

[mmenke, 2012/11/19 16:00:35]

Done - I agree.

 
   base::WeakPtrFactory<URLRequestRedirectJob> weak_factory_;
 };
 
 }  // namespace net
 
 #endif  // NET_URL_REQUEST_URL_REQUEST_REDIRECT_JOB_H_