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

Side by Side Diff: chrome/browser/safe_browsing/ping_manager.h

Issue 1117173004: Split cert reporter class into report building/serializing and sending (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: move //c/b/ssl classes into global namespace Created 5 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 CHROME_BROWSER_SAFE_BROWSING_PING_MANAGER_H_ 5 #ifndef CHROME_BROWSER_SAFE_BROWSING_PING_MANAGER_H_
6 #define CHROME_BROWSER_SAFE_BROWSING_PING_MANAGER_H_ 6 #define CHROME_BROWSER_SAFE_BROWSING_PING_MANAGER_H_
7 7
8 // A class that reports safebrowsing statistics to Google's SafeBrowsing 8 // A class that reports safebrowsing statistics to Google's SafeBrowsing
9 // servers. 9 // servers.
10 #include <set> 10 #include <set>
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
49 bool is_subresource, 49 bool is_subresource,
50 SBThreatType threat_type, 50 SBThreatType threat_type,
51 const std::string& post_data); 51 const std::string& post_data);
52 52
53 // Users can opt-in on the SafeBrowsing interstitial to send detailed 53 // Users can opt-in on the SafeBrowsing interstitial to send detailed
54 // malware reports. |report| is the serialized report. 54 // malware reports. |report| is the serialized report.
55 void ReportMalwareDetails(const std::string& report); 55 void ReportMalwareDetails(const std::string& report);
56 56
57 // Users can opt-in on the SSL interstitial to send reports of invalid 57 // Users can opt-in on the SSL interstitial to send reports of invalid
58 // certificate chains. 58 // certificate chains.
59 void ReportInvalidCertificateChain(const std::string& hostname, 59 void ReportInvalidCertificateChain(const std::string& serialized_report);
60 const net::SSLInfo& ssl_info);
61 60
62 void SetCertificateErrorReporterForTesting(scoped_ptr< 61 void SetCertificateErrorReporterForTesting(scoped_ptr<
63 chrome_browser_net::CertificateErrorReporter> certificate_error_reporter); 62 chrome_browser_net::CertificateErrorReporter> certificate_error_reporter);
64 63
65 private: 64 private:
66 FRIEND_TEST_ALL_PREFIXES(SafeBrowsingPingManagerTest, 65 FRIEND_TEST_ALL_PREFIXES(SafeBrowsingPingManagerTest,
67 TestSafeBrowsingHitUrl); 66 TestSafeBrowsingHitUrl);
68 FRIEND_TEST_ALL_PREFIXES(SafeBrowsingPingManagerTest, 67 FRIEND_TEST_ALL_PREFIXES(SafeBrowsingPingManagerTest,
69 TestMalwareDetailsUrl); 68 TestMalwareDetailsUrl);
70 69
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
102 Reports safebrowsing_reports_; 101 Reports safebrowsing_reports_;
103 102
104 // Sends reports of invalid SSL certificate chains. 103 // Sends reports of invalid SSL certificate chains.
105 scoped_ptr<chrome_browser_net::CertificateErrorReporter> 104 scoped_ptr<chrome_browser_net::CertificateErrorReporter>
106 certificate_error_reporter_; 105 certificate_error_reporter_;
107 106
108 DISALLOW_COPY_AND_ASSIGN(SafeBrowsingPingManager); 107 DISALLOW_COPY_AND_ASSIGN(SafeBrowsingPingManager);
109 }; 108 };
110 109
111 #endif // CHROME_BROWSER_SAFE_BROWSING_PING_MANAGER_H_ 110 #endif // CHROME_BROWSER_SAFE_BROWSING_PING_MANAGER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698