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

Unified Diff: chrome/browser/resources/settings/certificate_manager_page/certificate_list.js

Issue 2862123003: MD Settings: Certificates: Hide 'import and bind' in guest mode (Closed)
Patch Set: Created 3 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/resources/settings/certificate_manager_page/certificate_list.js
diff --git a/chrome/browser/resources/settings/certificate_manager_page/certificate_list.js b/chrome/browser/resources/settings/certificate_manager_page/certificate_list.js
index f13138a08f5e34a58f2f64a43202b8c1796a1c9f..33e9e46665b656660ee7f68c55a7cb91d6a7accc 100644
--- a/chrome/browser/resources/settings/certificate_manager_page/certificate_list.js
+++ b/chrome/browser/resources/settings/certificate_manager_page/certificate_list.js
@@ -18,6 +18,16 @@ Polymer({
/** @type {!CertificateType} */
certificateType: String,
+
+// <if expr="chromeos">
+ /** @private */
+ isGuest_: {
+ type: Boolean,
+ value: function() {
+ return loadTimeData.getBoolean('isGuest');
+ }
+ },
+// </if>
},
behaviors: [I18nBehavior],
@@ -58,7 +68,7 @@ Polymer({
* @private
*/
canImportAndBind_: function() {
- return this.certificateType == CertificateType.PERSONAL;
+ return !this.isGuest_ && this.certificateType == CertificateType.PERSONAL;
dpapad 2017/05/04 23:35:40 Optional: If you think this functionality is impor
stevenjb 2017/05/08 16:44:43 We don't currently have any "guest mode" tests tha
},
// </if>

Powered by Google App Engine
This is Rietveld 408576698