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

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

Issue 935663004: Add checkbox for reporting invalid TLS/SSL cert chains (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: use SecurityInterstitialMetricsHelper for extended reporting events Created 5 years, 9 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/safe_browsing/ui_manager.h
diff --git a/chrome/browser/safe_browsing/ui_manager.h b/chrome/browser/safe_browsing/ui_manager.h
index 543b1ab57284dfde1d7d3d16d439a511981e2625..0d2eb66b34d0b9f3e17253959a5641e5477ee011 100644
--- a/chrome/browser/safe_browsing/ui_manager.h
+++ b/chrome/browser/safe_browsing/ui_manager.h
@@ -26,6 +26,10 @@ namespace base {
class Thread;
}
+namespace net {
+class SSLInfo;
+}
+
// Construction needs to happen on the main thread.
class SafeBrowsingUIManager
: public base::RefCountedThreadSafe<SafeBrowsingUIManager> {
@@ -124,6 +128,10 @@ class SafeBrowsingUIManager
SBThreatType threat_type,
const std::string& post_data);
+ void ReportInvalidCertificateChain(const std::string& hostname,
mattm 2015/03/23 05:31:17 Add comment
estark 2015/03/23 16:42:13 Done.
+ const net::SSLInfo& ssl_info,
+ const base::Closure& callback);
+
// Add and remove observers. These methods must be invoked on the UI thread.
void AddObserver(Observer* observer);
void RemoveObserver(Observer* remove);
@@ -145,6 +153,9 @@ class SafeBrowsingUIManager
SBThreatType threat_type,
const std::string& post_data);
+ void ReportInvalidCertificateChainOnIOThread(const std::string& hostname,
+ const net::SSLInfo& ssl_info);
+
// Adds the given entry to the whitelist. Called on the UI thread.
void UpdateWhitelist(const UnsafeResource& resource);

Powered by Google App Engine
This is Rietveld 408576698