Move HSTS redirection from URLRequest to URLRequestHTTPJob.

net/url_request/url_request_http_job.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 NET_URL_REQUEST_URL_REQUEST_HTTP_JOB_H_
 #define NET_URL_REQUEST_URL_REQUEST_HTTP_JOB_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 22 matching lines @@
 
 class HttpRequestHeaders;
 class HttpResponseHeaders;
 class HttpResponseInfo;
 class HttpTransaction;
 class HttpUserAgentSettings;
 class ProxyInfo;
 class SSLPrivateKey;
 class UploadDataStream;
 class URLRequestContext;
+class URLRequestRedirectJob;

[mmenke, 2016/06/17 15:08:07]

Rather not forward declare this - can either make it return just a
URLRequestJob, or move the method into the CC file.

[Mike West, 2016/06/17 16:35:16]

Done.

 
 // A URLRequestJob subclass that is built on top of HttpTransaction. It
 // provides an implementation for both HTTP and HTTPS.
 class NET_EXPORT_PRIVATE URLRequestHttpJob : public URLRequestJob {
  public:
   static URLRequestJob* Factory(URLRequest* request,
                                 NetworkDelegate* network_delegate,
                                 const std::string& scheme);
 
  protected:
   URLRequestHttpJob(URLRequest* request,
                     NetworkDelegate* network_delegate,
                     const HttpUserAgentSettings* http_user_agent_settings);
 
   ~URLRequestHttpJob() override;
 
+  static URLRequestRedirectJob* MaybeInternallyRedirect(
+      URLRequest* request,
+      NetworkDelegate* network_delegate);

[mmenke, 2016/06/17 15:08:07]

Actually, if this is static, can we just move it into an anonymous namespace in
the CC file?

[Mike West, 2016/06/17 16:35:17]

Done.

+
   // Overridden from URLRequestJob:
   void SetPriority(RequestPriority priority) override;
   void Start() override;
   void Kill() override;
   void GetConnectionAttempts(ConnectionAttempts* out) const override;
 
   RequestPriority priority() const {
     return priority_;
   }
 
@@ skipping 210 matching lines @@
   int64_t total_sent_bytes_from_previous_transactions_;
 
   base::WeakPtrFactory<URLRequestHttpJob> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(URLRequestHttpJob);
 };
 
 }  // namespace net
 
 #endif  // NET_URL_REQUEST_URL_REQUEST_HTTP_JOB_H_