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

Unified Diff: chrome/browser/chrome_content_browser_client.cc

Issue 2543523002: Implement main CertificateReportingService code and add unit tests. (Closed)
Patch Set: jialiul comments 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
Index: chrome/browser/chrome_content_browser_client.cc
diff --git a/chrome/browser/chrome_content_browser_client.cc b/chrome/browser/chrome_content_browser_client.cc
index 1c7ce3b7eb720ea5a87af2bd1158e39a2da07c23..0c5402e3ecf559859775e5d260e361cd070815e8 100644
--- a/chrome/browser/chrome_content_browser_client.cc
+++ b/chrome/browser/chrome_content_browser_client.cc
@@ -670,7 +670,7 @@ class SafeBrowsingSSLCertReporter : public SSLCertReporter {
: safe_browsing_ui_manager_(safe_browsing_ui_manager) {}
~SafeBrowsingSSLCertReporter() override {}
- // SSLCertReporter implementation
+ // SSLCertReporter methods:
void ReportInvalidCertificateChain(
const std::string& serialized_report) override {
if (safe_browsing_ui_manager_) {
@@ -678,6 +678,7 @@ class SafeBrowsingSSLCertReporter : public SSLCertReporter {
serialized_report, base::Bind(&base::DoNothing));
}
}
+ void OnDidNotSendReport() override {}
private:
const scoped_refptr<safe_browsing::SafeBrowsingUIManager>

Powered by Google App Engine
This is Rietveld 408576698