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

Unified Diff: chrome/browser/ssl/security_state_model.h

Issue 1470813002: Add SecurityStateModelClient interface and implementation (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 1 month 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/ssl/security_state_model.h
diff --git a/chrome/browser/ssl/security_state_model.h b/chrome/browser/ssl/security_state_model.h
index 8989efb676d4dc249f84dd745deebe3db9b9db4d..155d52b1eeb951a36f7d59f02ec4bb1046048175 100644
--- a/chrome/browser/ssl/security_state_model.h
+++ b/chrome/browser/ssl/security_state_model.h
@@ -19,6 +19,7 @@ class WebContents;
} // namespace content
class Profile;
+class SecurityStateModelDelegate;
// SecurityStateModel provides high-level security information about a
// page or request. It is attached to a WebContents and will provide the
@@ -134,7 +135,8 @@ class SecurityStateModel
// given |profile|.
static void SecurityInfoForRequest(const GURL& url,
const content::SSLStatus& ssl,
- Profile* profile,
+ Profile* prfoile,
blundell 2015/11/23 16:06:17 nit: I liked the spelling better before ;).
estark 2015/11/23 16:23:39 Doh. Done.
+ bool used_known_mitm_certificate,
SecurityInfo* security_info);
private:
@@ -151,6 +153,8 @@ class SecurityStateModel
mutable GURL visible_url_;
mutable content::SSLStatus visible_ssl_status_;
+ scoped_ptr<SecurityStateModelDelegate> delegate_;
blundell 2015/11/23 16:06:17 nit: Let's add a comment here explaining how this
estark 2015/11/23 16:23:40 Done.
+
DISALLOW_COPY_AND_ASSIGN(SecurityStateModel);
};

Powered by Google App Engine
This is Rietveld 408576698