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

Unified Diff: chrome/browser/ui/website_settings/website_settings.h

Issue 473643002: Add button to page info to revoke user certificate decisions. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 4 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/ui/website_settings/website_settings.h
diff --git a/chrome/browser/ui/website_settings/website_settings.h b/chrome/browser/ui/website_settings/website_settings.h
index 8bbf48188e2c41fffb35eb8ab8e6b32ba8e7b7cc..5b571e6165262ab01d24a389546f608beb7e5e7a 100644
--- a/chrome/browser/ui/website_settings/website_settings.h
+++ b/chrome/browser/ui/website_settings/website_settings.h
@@ -22,6 +22,7 @@ class CertStore;
struct SSLStatus;
}
+class ChromeSSLHostStateDelegate;
class InfoBarService;
class HostContentSettingsMap;
class Profile;
@@ -97,6 +98,8 @@ class WebsiteSettings : public TabSpecificContentSettings::SiteDataObserver {
return site_connection_status_;
}
+ const GURL& site_url() const { return site_url_; }
+
SiteIdentityStatus site_identity_status() const {
return site_identity_status_;
}
@@ -113,6 +116,10 @@ class WebsiteSettings : public TabSpecificContentSettings::SiteDataObserver {
return organization_name_;
}
+ ChromeSSLHostStateDelegate* chrome_ssl_host_state_delegate() {
+ return chrome_ssl_host_state_delegate_;
+ }
+
// SiteDataObserver implementation.
virtual void OnSiteDataAccessed() OVERRIDE;
@@ -178,6 +185,14 @@ class WebsiteSettings : public TabSpecificContentSettings::SiteDataObserver {
// This string will be displayed in the UI.
base::string16 site_identity_details_;
+ // Set when the user has explicitly bypassed an SSL error for this host or
+ // explicitly denied it (the latter of which is not currently possible in the
+ // Chrome UI) and has a flag set to remember ssl decisions (explicit flag or
+ // in the experimental group). When |show_ssl_decision_revoke_button| is
+ // true, the connection area of the page info will include an option for the
+ // user to revoke their decision to bypass the SSL error for this host.
+ bool show_ssl_decision_revoke_button_;
+
// Details about the connection to the website. In case of an encrypted
// connection |site_connection_details_| contains encryption details, like
// encryption strength and ssl protocol version. This string will be
@@ -200,6 +215,10 @@ class WebsiteSettings : public TabSpecificContentSettings::SiteDataObserver {
// Used to request the number of page visits.
base::CancelableTaskTracker visit_count_task_tracker_;
+ // Service for managing SSL error page bypasses. Used to revoke bypass
+ // decisions by users.
+ ChromeSSLHostStateDelegate* chrome_ssl_host_state_delegate_;
+
DISALLOW_COPY_AND_ASSIGN(WebsiteSettings);
};

Powered by Google App Engine
This is Rietveld 408576698