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

Unified Diff: chrome/browser/safe_browsing/ping_manager.h

Issue 2578873003: Remove certificate reporting code from SafeBrowsing ping and ui managers (Closed)
Patch Set: Rebase Created 4 years 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/browser/safe_browsing/ping_manager.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/safe_browsing/ping_manager.h
diff --git a/chrome/browser/safe_browsing/ping_manager.h b/chrome/browser/safe_browsing/ping_manager.h
index ded4afabb7ab9660849747f2358dcbdc5382493b..9f0c1977ce8617790536e7b4ad1326e04f01ae35 100644
--- a/chrome/browser/safe_browsing/ping_manager.h
+++ b/chrome/browser/safe_browsing/ping_manager.h
@@ -23,10 +23,6 @@
#include "net/url_request/url_fetcher_delegate.h"
#include "url/gurl.h"
-namespace certificate_reporting {
-class ErrorReporter;
-}
-
namespace net {
class URLRequestContextGetter;
} // namespace net
@@ -56,14 +52,6 @@ class SafeBrowsingPingManager : public net::URLFetcherDelegate {
// threat reports. |report| is the serialized report.
void ReportThreatDetails(const std::string& report);
- // Users can opt-in on the SSL interstitial to send reports of invalid
- // certificate chains.
- void ReportInvalidCertificateChain(const std::string& serialized_report);
-
- void SetCertificateErrorReporterForTesting(
- std::unique_ptr<certificate_reporting::ErrorReporter>
- certificate_error_reporter);
-
// Report permission action to SafeBrowsing servers.
void ReportPermissionAction(const PermissionReportInfo& report_info);
@@ -111,10 +99,6 @@ class SafeBrowsingPingManager : public net::URLFetcherDelegate {
// We add both "hit" and "detail" fetchers in this set.
Reports safebrowsing_reports_;
- // Sends reports of invalid SSL certificate chains.
- std::unique_ptr<certificate_reporting::ErrorReporter>
- certificate_error_reporter_;
-
// Sends reports of permission actions.
std::unique_ptr<PermissionReporter> permission_reporter_;
« no previous file with comments | « no previous file | chrome/browser/safe_browsing/ping_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698