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

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

Issue 1150173002: Remove the SCT viewer UI (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Review feedback Created 5 years, 7 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/views/website_settings/website_settings_popup_view.cc
diff --git a/chrome/browser/ui/views/website_settings/website_settings_popup_view.cc b/chrome/browser/ui/views/website_settings/website_settings_popup_view.cc
index 65d281dcb4e1e383d902b928d92051e89ebe1e36..6fe93a39499a09b661534b3ffcc959ce93dba1d6 100644
--- a/chrome/browser/ui/views/website_settings/website_settings_popup_view.cc
+++ b/chrome/browser/ui/views/website_settings/website_settings_popup_view.cc
@@ -312,7 +312,6 @@ WebsiteSettingsPopupView::WebsiteSettingsPopupView(
connection_tab_(nullptr),
identity_info_content_(nullptr),
certificate_dialog_link_(nullptr),
- signed_certificate_timestamps_link_(nullptr),
reset_decisions_button_(nullptr),
cert_id_(0),
help_center_link_(nullptr),
@@ -575,21 +574,11 @@ void WebsiteSettingsPopupView::SetIdentityInfo(
base::string16 headline;
if (identity_info.cert_id) {
cert_id_ = identity_info.cert_id;
- signed_certificate_timestamp_ids_.assign(
- identity_info.signed_certificate_timestamp_ids.begin(),
- identity_info.signed_certificate_timestamp_ids.end());
certificate_dialog_link_ = new views::Link(
l10n_util::GetStringUTF16(IDS_PAGEINFO_CERT_INFO_BUTTON));
certificate_dialog_link_->set_listener(this);
- if (!signed_certificate_timestamp_ids_.empty()) {
- signed_certificate_timestamps_link_ =
- new views::Link(l10n_util::GetStringUTF16(
- IDS_PAGEINFO_CERT_TRANSPARENCY_INFO_BUTTON));
- signed_certificate_timestamps_link_->set_listener(this);
- }
-
if (identity_info.show_ssl_decision_revoke_button) {
reset_decisions_button_ = new views::LabelButton(
this,
@@ -606,7 +595,6 @@ void WebsiteSettingsPopupView::SetIdentityInfo(
base::string16(), // The identity section has no headline.
base::UTF8ToUTF16(identity_info.identity_status_description),
certificate_dialog_link_,
- signed_certificate_timestamps_link_,
reset_decisions_button_);
ResetConnectionSection(
@@ -615,7 +603,6 @@ void WebsiteSettingsPopupView::SetIdentityInfo(
base::string16(), // The connection section has no headline.
base::UTF8ToUTF16(identity_info.connection_status_description),
nullptr,
- nullptr,
nullptr);
connection_tab_->InvalidateLayout();
@@ -631,7 +618,6 @@ void WebsiteSettingsPopupView::SetFirstVisit(
l10n_util::GetStringUTF16(IDS_PAGE_INFO_SITE_INFO_TITLE),
first_visit,
nullptr,
- nullptr,
nullptr);
connection_tab_->InvalidateLayout();
Layout();
@@ -746,7 +732,6 @@ void WebsiteSettingsPopupView::ResetConnectionSection(
const base::string16& headline,
const base::string16& text,
views::Link* link,
- views::Link* secondary_link,
views::LabelButton* reset_decisions_button) {
section_container->RemoveAllChildViews(true);
@@ -815,11 +800,6 @@ void WebsiteSettingsPopupView::ResetConnectionSection(
content_layout->AddView(link);
}
- if (secondary_link) {
- content_layout->StartRow(1, 0);
- content_layout->AddView(secondary_link);
- }
-
if (reset_decisions_button) {
content_layout->StartRow(1, 0);
content_layout->AddView(reset_decisions_button);
@@ -844,11 +824,6 @@ void WebsiteSettingsPopupView::HandleLinkClickedAsync(views::Link* source) {
presenter_->RecordWebsiteSettingsAction(
WebsiteSettings::WEBSITE_SETTINGS_CERTIFICATE_DIALOG_OPENED);
ShowCertificateViewerByID(web_contents_, parent, cert_id_);
- } else if (source == signed_certificate_timestamps_link_) {
- chrome::ShowSignedCertificateTimestampsViewer(
- web_contents_, signed_certificate_timestamp_ids_);
- presenter_->RecordWebsiteSettingsAction(
- WebsiteSettings::WEBSITE_SETTINGS_TRANSPARENCY_VIEWER_OPENED);
} else if (source == help_center_link_) {
browser_->OpenURL(
content::OpenURLParams(GURL(chrome::kPageInfoHelpCenterURL),

Powered by Google App Engine
This is Rietveld 408576698