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

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

Issue 465133004: Remove DenyCertForHost from SSLHostStateDelegate API. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Sleevi comments Created 6 years, 3 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.cc
diff --git a/chrome/browser/ui/website_settings/website_settings.cc b/chrome/browser/ui/website_settings/website_settings.cc
index c895a5c73b47aab5bbbbe64593e630cbbdca03f6..dfde2a01679d36c88d78d6297494eff5ac1c4ffe 100644
--- a/chrome/browser/ui/website_settings/website_settings.cc
+++ b/chrome/browser/ui/website_settings/website_settings.cc
@@ -328,7 +328,8 @@ void WebsiteSettings::OnUIClosing() {
void WebsiteSettings::OnRevokeSSLErrorBypassButtonPressed() {
DCHECK(chrome_ssl_host_state_delegate_);
- chrome_ssl_host_state_delegate_->RevokeUserDecisionsHard(site_url().host());
+ chrome_ssl_host_state_delegate_->RevokeUserAllowExceptionsHard(
+ site_url().host());
did_revoke_user_ssl_decisions_ = true;
}
@@ -582,8 +583,8 @@ void WebsiteSettings::Init(Profile* profile,
// Only show an SSL decision revoke button if both the user has chosen to
// bypass SSL host errors for this host in the past and the user is not using
// the traditional "forget-at-session-restart" error decision memory.
- show_ssl_decision_revoke_button_ = delegate->HasUserDecision(url.host()) &&
- InRememberCertificateErrorDecisionsGroup();
+ show_ssl_decision_revoke_button_ = delegate->HasAllowException(url.host()) &&
+ InRememberCertificateErrorDecisionsGroup();
// By default select the permissions tab that displays all the site
// permissions. In case of a connection error or an issue with the

Powered by Google App Engine
This is Rietveld 408576698