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

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

Issue 2627143003: Remove message about TLS renegotiation info extension. (Closed)
Patch Set: Created 3 years, 11 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
« no previous file with comments | « no previous file | components/pageinfo_strings.grdp » ('j') | ios/chrome/browser/ui/omnibox/page_info_model.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 c9dae32d9c4bf20c068e3461332cc62e0b5aba7c..79316ed1b039f40a00541728ca7f6001305742c7 100644
--- a/chrome/browser/ui/website_settings/website_settings.cc
+++ b/chrome/browser/ui/website_settings/website_settings.cc
@@ -617,9 +617,6 @@ void WebsiteSettings::Init(const GURL& url,
IDS_PAGE_INFO_SECURITY_TAB_SSL_VERSION,
ASCIIToUTF16(ssl_version_str));
- bool no_renegotiation =
- (security_info.connection_status &
- net::SSL_CONNECTION_NO_RENEGOTIATION_EXTENSION) != 0;
const char *key_exchange, *cipher, *mac;
bool is_aead, is_tls13;
net::SSLCipherSuiteToStrings(&key_exchange, &cipher, &mac, &is_aead,
@@ -650,12 +647,6 @@ void WebsiteSettings::Init(const GURL& url,
SITE_CONNECTION_STATUS_INSECURE_PASSIVE_SUBRESOURCE) {
site_connection_status_ = SITE_CONNECTION_STATUS_ENCRYPTED_ERROR;
}
-
- if (no_renegotiation) {
- site_connection_details_ += ASCIIToUTF16("\n\n");
- site_connection_details_ += l10n_util::GetStringUTF16(
- IDS_PAGE_INFO_SECURITY_TAB_RENEGOTIATION_MESSAGE);
- }
}
// Check if a user decision has been made to allow or deny certificates with
« no previous file with comments | « no previous file | components/pageinfo_strings.grdp » ('j') | ios/chrome/browser/ui/omnibox/page_info_model.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698