OLD | NEW |
---|---|
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2011 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 CHROME_BROWSER_NET_CHROME_FRAUDULENT_CERTIFICATE_REPORTER_H_ | 5 #ifndef CHROME_BROWSER_NET_CHROME_FRAUDULENT_CERTIFICATE_REPORTER_H_ |
6 #define CHROME_BROWSER_NET_CHROME_FRAUDULENT_CERTIFICATE_REPORTER_H_ | 6 #define CHROME_BROWSER_NET_CHROME_FRAUDULENT_CERTIFICATE_REPORTER_H_ |
7 | 7 |
8 #include <set> | 8 #include <set> |
9 #include <string> | 9 #include <string> |
10 | 10 |
11 #include "chrome/browser/net/certificate_error_reporter.h" | |
Ryan Sleevi
2015/03/06 21:32:47
You can forward declare this class
estark
2015/03/07 00:38:50
I don't think I can, because of the scoped_ptr to
| |
11 #include "net/url_request/fraudulent_certificate_reporter.h" | 12 #include "net/url_request/fraudulent_certificate_reporter.h" |
12 #include "net/url_request/url_request.h" | 13 #include "net/url_request/url_request.h" |
Ryan Sleevi
2015/03/06 21:32:47
You no longer need this include
estark
2015/03/07 00:38:50
Done.
| |
13 | 14 |
14 namespace net { | 15 namespace net { |
15 class URLRequestContext; | 16 class URLRequestContext; |
16 } | 17 } |
17 | 18 |
18 namespace chrome_browser_net { | 19 namespace chrome_browser_net { |
19 | 20 |
20 class ChromeFraudulentCertificateReporter | 21 class ChromeFraudulentCertificateReporter |
21 : public net::FraudulentCertificateReporter, | 22 : public net::FraudulentCertificateReporter { |
22 public net::URLRequest::Delegate { | |
23 public: | 23 public: |
24 explicit ChromeFraudulentCertificateReporter( | 24 explicit ChromeFraudulentCertificateReporter( |
25 net::URLRequestContext* request_context); | 25 net::URLRequestContext* request_context); |
26 | 26 |
27 ~ChromeFraudulentCertificateReporter() override; | 27 // Useful for tests to use a mock reporter. |
28 explicit ChromeFraudulentCertificateReporter( | |
29 scoped_ptr<CertificateErrorReporter> certificate_reporter); | |
Ryan Sleevi
2015/03/06 21:32:47
You need to include scoped_ptr's header
estark
2015/03/07 00:38:50
Done.
| |
28 | 30 |
29 // Allows users of this class to override this and set their own URLRequest | 31 ~ChromeFraudulentCertificateReporter() override {} |
30 // type. Used by SendReport. | |
31 virtual scoped_ptr<net::URLRequest> CreateURLRequest( | |
32 net::URLRequestContext* context); | |
33 | 32 |
34 // net::FraudulentCertificateReporter | 33 // net::FraudulentCertificateReporter |
35 void SendReport(const std::string& hostname, | 34 void SendReport(const std::string& hostname, |
36 const net::SSLInfo& ssl_info) override; | 35 const net::SSLInfo& ssl_info) override; |
37 | 36 |
38 // net::URLRequest::Delegate | |
39 void OnResponseStarted(net::URLRequest* request) override; | |
40 void OnReadCompleted(net::URLRequest* request, int bytes_read) override; | |
41 | |
42 protected: | |
43 net::URLRequestContext* const request_context_; | |
44 | |
45 private: | 37 private: |
46 // Performs post-report cleanup. | 38 scoped_ptr<CertificateErrorReporter> certificate_reporter_; |
47 void RequestComplete(net::URLRequest* request); | |
48 | |
49 const GURL upload_url_; | |
50 // Owns the contained requests. | |
51 std::set<net::URLRequest*> inflight_requests_; | |
52 | 39 |
53 DISALLOW_COPY_AND_ASSIGN(ChromeFraudulentCertificateReporter); | 40 DISALLOW_COPY_AND_ASSIGN(ChromeFraudulentCertificateReporter); |
54 }; | 41 }; |
55 | 42 |
56 } // namespace chrome_browser_net | 43 } // namespace chrome_browser_net |
57 | 44 |
58 #endif // CHROME_BROWSER_NET_CHROME_FRAUDULENT_CERTIFICATE_REPORTER_H_ | 45 #endif // CHROME_BROWSER_NET_CHROME_FRAUDULENT_CERTIFICATE_REPORTER_H_ |
59 | |
OLD | NEW |