Replaced static URLRequestHTTPJob factory with non-static protocol handler for HTTP jobs.

net/url_request/http_protocol_handler.h

Index: net/url_request/http_protocol_handler.h
diff --git a/net/url_request/http_protocol_handler.h b/net/url_request/http_protocol_handler.h
new file mode 100644
index 0000000000000000000000000000000000000000..c8d2f3c176a264baaaa0a7499214894181771d17
--- /dev/null
+++ b/net/url_request/http_protocol_handler.h
@@ -0,0 +1,56 @@
+// 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_HTTP_PROTOCOL_HANDLER_H_
+#define NET_URL_REQUEST_HTTP_PROTOCOL_HANDLER_H_
+

[mmenke, 2012/07/18 16:21:53]

#include <string>

[shalev, 2012/07/23 21:00:21]

Done.

+#include "base/basictypes.h"
+#include "base/compiler_specific.h"
+#include "net/url_request/url_request_job_factory.h"
+
+namespace net {
+
+class CookieStore;
+class FraudulentCertificateReporter;
+class HttpTransactionFactory;
+class NetworkDelegate;
+class SSLConfigService;
+class TransportSecurityState;
+class URLRequestJob;
+class URLRequestThrottlerManager;
+
+// Implements a ProtocolHandler for HTTP.
+class NET_EXPORT HttpProtocolHandler :
+    public URLRequestJobFactory::ProtocolHandler {
+ public:
+  HttpProtocolHandler(
+      HttpTransactionFactory* http_transaction_factory,
+      NetworkDelegate* network_delegate,
+      URLRequestThrottlerManager* throttler_manager,
+      const std::string& accept_language,
+      const std::string& accept_charset,
+      CookieStore* cookie_store,
+      FraudulentCertificateReporter* fraudulent_certificate_reporter,
+      SSLConfigService* ssl_config_service,
+      TransportSecurityState* transport_security_state);
+
+  virtual URLRequestJob* MaybeCreateJob(URLRequest* request) const OVERRIDE;
+
+ private:
+  HttpTransactionFactory* http_transaction_factory_;
+  NetworkDelegate* network_delegate_;
+  URLRequestThrottlerManager* throttler_manager_;
+  std::string accept_language_;
+  std::string accept_charset_;

[mmenke, 2012/07/18 16:21:53]

nit:  Might as well make both these strings const.

[shalev, 2012/07/23 21:00:21]

Done.

+  CookieStore* cookie_store_;
+  FraudulentCertificateReporter* fraudulent_certificate_reporter_;
+  SSLConfigService* ssl_config_service_;
+  TransportSecurityState* transport_security_state_;
+
+  DISALLOW_COPY_AND_ASSIGN(HttpProtocolHandler);
+};
+
+}  // namespace net
+
+#endif  // NET_URL_REQUEST_HTTP_PROTOCOL_HANDLER_H_