diff options
author | markusheintz@chromium.org <markusheintz@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-04-25 09:57:41 +0000 |
---|---|---|
committer | markusheintz@chromium.org <markusheintz@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-04-25 09:57:41 +0000 |
commit | 24c8818cc2546b92de0e7b2cdc95eba51bd99379 (patch) | |
tree | db44c21d4a4619fcc5bb437bdb63551a0665b8dd /chrome/browser/website_settings.cc | |
parent | 31b7610edd7f86ada422938430dc5feed12cfd0c (diff) | |
download | chromium_src-24c8818cc2546b92de0e7b2cdc95eba51bd99379.zip chromium_src-24c8818cc2546b92de0e7b2cdc95eba51bd99379.tar.gz chromium_src-24c8818cc2546b92de0e7b2cdc95eba51bd99379.tar.bz2 |
Display site permissions in a tab and add a tab to display information about the site's identity and the site's connection.
UI string changes: https://chromiumcodereview.appspot.com/10180002/
BUG=113688
TEST=WebsiteSettingsTest*
Review URL: https://chromiumcodereview.appspot.com/10163002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@133886 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/website_settings.cc')
-rw-r--r-- | chrome/browser/website_settings.cc | 43 |
1 files changed, 21 insertions, 22 deletions
diff --git a/chrome/browser/website_settings.cc b/chrome/browser/website_settings.cc index 5850576..d63ccf9 100644 --- a/chrome/browser/website_settings.cc +++ b/chrome/browser/website_settings.cc @@ -84,31 +84,10 @@ WebsiteSettings::WebsiteSettings( content_settings_(profile->GetHostContentSettingsMap()) { ui_->SetPresenter(this); Init(profile, url, ssl); - // After initialization the status about the site's connection - // and it's identity must be available. - DCHECK_NE(site_identity_status_, SITE_IDENTITY_STATUS_UNKNOWN); - DCHECK_NE(site_connection_status_, SITE_CONNECTION_STATUS_UNKNOWN); - - // TODO(markusheintz): Add the strings below to the grd file once a decision - // has been made about the exact wording. - std::string site_info; - switch (site_identity_status_) { - case WebsiteSettings::SITE_IDENTITY_STATUS_CERT: - case WebsiteSettings::SITE_IDENTITY_STATUS_DNSSEC_CERT: - site_info = "Identity verified"; - break; - case WebsiteSettings::SITE_IDENTITY_STATUS_EV_CERT: - site_info = UTF16ToUTF8(organization_name()); - site_info += " - Identity verified"; - break; - default: - site_info = "Identity not verified"; - break; - } - ui_->SetSiteInfo(site_info); PresentSitePermissions(); PresentSiteData(); + PresentSiteIdentity(); } WebsiteSettings::~WebsiteSettings() { @@ -445,6 +424,26 @@ void WebsiteSettings::PresentSiteData() { ui_->SetCookieInfo(cookie_info_list); } +void WebsiteSettings::PresentSiteIdentity() { + // After initialization the status about the site's connection + // and it's identity must be available. + DCHECK_NE(site_identity_status_, SITE_IDENTITY_STATUS_UNKNOWN); + DCHECK_NE(site_connection_status_, SITE_CONNECTION_STATUS_UNKNOWN); + WebsiteSettingsUI::IdentityInfo info; + if (site_identity_status_ == SITE_IDENTITY_STATUS_EV_CERT) + info.site_identity = UTF16ToUTF8(organization_name()); + else + info.site_identity = site_url_.host(); + + info.connection_status = site_connection_status_; + info.connection_status_description = + UTF16ToUTF8(site_connection_details_); + info.identity_status = site_identity_status_; + info.identity_status_description = + UTF16ToUTF8(site_identity_details_); + ui_->SetIdentityInfo(info); +} + // static void WebsiteSettings::Show(gfx::NativeWindow parent, Profile* profile, |