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

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

Issue 1819563002: MD Settings: Certificate manager, populate UI. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Tests. Created 4 years, 9 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_manager_page.js
diff --git a/chrome/browser/resources/settings/certificate_manager_page/certificate_manager_page.js b/chrome/browser/resources/settings/certificate_manager_page/certificate_manager_page.js
index 14979a302f4574ffb43b6f62d2cd9b06f76ffb54..2d8109e55ed055fdc40b1966bd9de3b0c7f0bee1 100644
--- a/chrome/browser/resources/settings/certificate_manager_page/certificate_manager_page.js
+++ b/chrome/browser/resources/settings/certificate_manager_page/certificate_manager_page.js
@@ -3,18 +3,67 @@
// found in the LICENSE file.
/**
- * @fileoverview
- * 'settings-certificate-manager-page' is the settings page containing SSL
- * certificate settings.
- *
- * Example:
- *
- * <iron-animated-pages>
- * <settings-certificate-manager-page prefs="{{prefs}}">
- * </settings-certificate-manager-page>
- * ... other pages ...
- * </iron-animated-pages>
+ * @fileoverview 'settings-certificate-manager-page' is the settings page
+ * containing SSL certificate settings.
*/
Polymer({
is: 'settings-certificate-manager-page',
+
+ behaviors: [WebUIListenerBehavior],
+
+ properties: {
+ /** @type {number} */
+ selected: {
+ type: Number,
+ value: 0,
+ },
+
+ /** @type {!Array<!Certificate>} */
+ personalCerts: {
+ type: Array,
+ value: function() { return []; },
+ },
+
+ /** @type {!Array<!Certificate>} */
+ serverCerts: {
+ type: Array,
+ value: function() { return []; },
+ },
+
+ /** @type {!Array<!Certificate>} */
+ caCerts: {
+ type: Array,
+ value: function() { return []; },
+ },
+
+ /** @type {!Array<!Certificate>} */
+ otherCerts: {
+ type: Array,
+ value: function() { return []; },
+ },
+ },
+
+ /** @private {!settings.CertificatesManagerBrowserProxy} */
+ browserProxy_: null,
Dan Beam 2016/03/22 02:08:28 nit: if you only use this once, it may be cheaper/
dpapad 2016/03/22 18:07:02 Done.
+
+ /** @override */
+ created: function() {
+ this.browserProxy_ = settings.CertificatesBrowserProxyImpl.getInstance();
+ },
+
+ /** @override */
+ attached: function() {
+ this.addWebUIListener('certificates-changed', this.set.bind(this));
+ this.browserProxy_.refreshCertificates();
+ },
+
+ /**
+ * @param {number} selectedIndex
+ * @param {number} tabIndex
+ * @return {boolean} Whether to show tab at |tabIndex|.
+ * @private
+ */
+ isTabSelected_: function(selectedIndex, tabIndex) {
+ return selectedIndex == tabIndex;
+ },
});

Powered by Google App Engine
This is Rietveld 408576698