Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(417)

Side by Side Diff: net/url_request/url_request_http_job.h

Issue 1888963004: Add HttpProtocolHandler and convert everything to use it (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@remove-supports-scheme
Patch Set: more addressing comments Created 4 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef NET_URL_REQUEST_URL_REQUEST_HTTP_JOB_H_ 5 #ifndef NET_URL_REQUEST_URL_REQUEST_HTTP_JOB_H_
6 #define NET_URL_REQUEST_URL_REQUEST_HTTP_JOB_H_ 6 #define NET_URL_REQUEST_URL_REQUEST_HTTP_JOB_H_
7 7
8 #include <stddef.h> 8 #include <stddef.h>
9 #include <stdint.h> 9 #include <stdint.h>
10 10
(...skipping 27 matching lines...) Expand all
38 class HttpUserAgentSettings; 38 class HttpUserAgentSettings;
39 class ProxyInfo; 39 class ProxyInfo;
40 class SSLPrivateKey; 40 class SSLPrivateKey;
41 class UploadDataStream; 41 class UploadDataStream;
42 class URLRequestContext; 42 class URLRequestContext;
43 43
44 // A URLRequestJob subclass that is built on top of HttpTransaction. It 44 // A URLRequestJob subclass that is built on top of HttpTransaction. It
45 // provides an implementation for both HTTP and HTTPS. 45 // provides an implementation for both HTTP and HTTPS.
46 class NET_EXPORT_PRIVATE URLRequestHttpJob : public URLRequestJob { 46 class NET_EXPORT_PRIVATE URLRequestHttpJob : public URLRequestJob {
47 public: 47 public:
48 static URLRequestJob* Factory(URLRequest* request, 48 // Creates a URLRequestHttpJob. Non-test code other than HttpProtocolHandler
49 NetworkDelegate* network_delegate, 49 // shouldn't use this constructor directly, because HttpProtocolHandler is
50 const std::string& scheme); 50 // where HSTS redirects happen.
51
52 protected:
53 URLRequestHttpJob(URLRequest* request, 51 URLRequestHttpJob(URLRequest* request,
54 NetworkDelegate* network_delegate, 52 NetworkDelegate* network_delegate,
55 const HttpUserAgentSettings* http_user_agent_settings); 53 const HttpUserAgentSettings* http_user_agent_settings);
56 54
55 protected:
57 ~URLRequestHttpJob() override; 56 ~URLRequestHttpJob() override;
58 57
59 // Overridden from URLRequestJob: 58 // Overridden from URLRequestJob:
60 void SetPriority(RequestPriority priority) override; 59 void SetPriority(RequestPriority priority) override;
61 void Start() override; 60 void Start() override;
62 void Kill() override; 61 void Kill() override;
63 void GetConnectionAttempts(ConnectionAttempts* out) const override; 62 void GetConnectionAttempts(ConnectionAttempts* out) const override;
64 63
65 RequestPriority priority() const { 64 RequestPriority priority() const {
66 return priority_; 65 return priority_;
(...skipping 214 matching lines...) Expand 10 before | Expand all | Expand 10 after
281 int64_t total_sent_bytes_from_previous_transactions_; 280 int64_t total_sent_bytes_from_previous_transactions_;
282 281
283 base::WeakPtrFactory<URLRequestHttpJob> weak_factory_; 282 base::WeakPtrFactory<URLRequestHttpJob> weak_factory_;
284 283
285 DISALLOW_COPY_AND_ASSIGN(URLRequestHttpJob); 284 DISALLOW_COPY_AND_ASSIGN(URLRequestHttpJob);
286 }; 285 };
287 286
288 } // namespace net 287 } // namespace net
289 288
290 #endif // NET_URL_REQUEST_URL_REQUEST_HTTP_JOB_H_ 289 #endif // NET_URL_REQUEST_URL_REQUEST_HTTP_JOB_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698