Add checkbox for reporting invalid TLS/SSL cert chains

chrome/browser/safe_browsing/ui_manager.cc

Index: chrome/browser/safe_browsing/ui_manager.cc
diff --git a/chrome/browser/safe_browsing/ui_manager.cc b/chrome/browser/safe_browsing/ui_manager.cc
index 45728d35409b0ae387b9f42c187e36049775c98e..1ccd79da75787c57927116216eb5878d1272328d 100644
--- a/chrome/browser/safe_browsing/ui_manager.cc
+++ b/chrome/browser/safe_browsing/ui_manager.cc
@@ -26,6 +26,7 @@
 #include "content/public/browser/notification_service.h"
 #include "content/public/browser/web_contents.h"
 #include "net/base/registry_controlled_domains/registry_controlled_domain.h"
+#include "net/ssl/ssl_info.h"
 #include "net/url_request/url_request_context.h"
 #include "net/url_request/url_request_context_getter.h"
 
@@ -213,6 +214,19 @@ void SafeBrowsingUIManager::ReportSafeBrowsingHit(
                  threat_type, post_data));
 }
 
+void SafeBrowsingUIManager::ReportInvalidCertificateChain(
+    const std::string& hostname,
+    const net::SSLInfo& ssl_info,
+    const base::Closure& callback) {
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+  BrowserThread::PostTaskAndReply(
+      BrowserThread::IO, FROM_HERE,
+      base::Bind(
+          &SafeBrowsingUIManager::ReportInvalidCertificateChainOnIOThread, this,
+          hostname, ssl_info),
+      callback);
+}
+
 void SafeBrowsingUIManager::AddObserver(Observer* observer) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   observer_list_.AddObserver(observer);
@@ -246,6 +260,14 @@ void SafeBrowsingUIManager::ReportSafeBrowsingHitOnIOThread(
       threat_type, post_data);
 }
 
+void SafeBrowsingUIManager::ReportInvalidCertificateChainOnIOThread(
+    const std::string& hostname,
+    const net::SSLInfo& ssl_info) {
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
+  sb_service_->ping_manager()->ReportInvalidCertificateChain(hostname,

[mattm, 2015/03/23 05:31:17]

This should have null checks like in SendSerializedMalwareDetails

[estark, 2015/03/23 16:42:13]

Done.

+                                                             ssl_info);
+}
+
 // If the user had opted-in to send MalwareDetails, this gets called
 // when the report is ready.
 void SafeBrowsingUIManager::SendSerializedMalwareDetails(
@@ -305,4 +327,3 @@ bool SafeBrowsingUIManager::IsWhitelisted(const UnsafeResource& resource) {
   }
   return false;
 }
-