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

Unified Diff: components/toolbar/toolbar_model_impl.cc

Issue 2119033002: [Material][Mac] Implement Omnibox Verbose State Chips (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix for rsesek Created 4 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: components/toolbar/toolbar_model_impl.cc
diff --git a/components/toolbar/toolbar_model_impl.cc b/components/toolbar/toolbar_model_impl.cc
index e7d693e698e5c3e1c43eb599d4fab8161e6d4de8..287033313b6e1acc775ba0e490f860d264f95831 100644
--- a/components/toolbar/toolbar_model_impl.cc
+++ b/components/toolbar/toolbar_model_impl.cc
@@ -129,6 +129,21 @@ base::string16 ToolbarModelImpl::GetEVCertName() const {
base::UTF8ToUTF16(cert->subject().country_name));
}
+base::string16 ToolbarModelImpl::GetSecurityVerboseText() const {
+ switch (GetSecurityLevel(false)) {
+ case SecurityStateModel::SECURE:
+ return l10n_util::GetStringUTF16(IDS_SECURE_VERBOSE_STATE);
+ case SecurityStateModel::EV_SECURE:
+ return GetEVCertName();
+ case SecurityStateModel::SECURITY_ERROR:
+ return l10n_util::GetStringUTF16(IDS_NOT_SECURE_VERBOSE_STATE);
+ case SecurityStateModel::NONE:
Peter Kasting 2016/08/18 21:15:00 Nit: I'm inclined to just change all these to a "d
spqchan 2016/08/18 21:56:32 Done.
+ case SecurityStateModel::SECURITY_WARNING:
+ case SecurityStateModel::SECURITY_POLICY_WARNING:
+ return base::string16();
+ }
+}
+
bool ToolbarModelImpl::ShouldDisplayURL() const {
return delegate_->ShouldDisplayURL();
}

Powered by Google App Engine
This is Rietveld 408576698