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

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

Issue 1784553004: MD Settings: Certificate manager, delete confirmation dialog. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@certificates_catrustedit
Patch Set: Addressing comments. 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/certificates_browser_proxy.js
diff --git a/chrome/browser/resources/settings/certificate_manager_page/certificates_browser_proxy.js b/chrome/browser/resources/settings/certificate_manager_page/certificates_browser_proxy.js
index 26d6749fe15969a15b127bdbd495378e26303df3..cf2cbe4f135950f00a2b81210163e1dffd1d6e2e 100644
--- a/chrome/browser/resources/settings/certificate_manager_page/certificates_browser_proxy.js
+++ b/chrome/browser/resources/settings/certificate_manager_page/certificates_browser_proxy.js
@@ -64,6 +64,17 @@ var CertificatesError;
var CertificatesImportError;
cr.define('settings', function() {
+ /**
+ * Enumeration of all possible certificate types.
+ * @enum {string}
+ */
+ var CertificateType = {
+ CA: 'ca',
+ OTHER: 'other',
+ PERSONAL: 'personal',
+ SERVER: 'server',
+ };
+
/** @interface */
function CertificatesBrowserProxy() {}
@@ -83,7 +94,8 @@ cr.define('settings', function() {
/**
* @param {string} id
- * @return {!Promise}
+ * @return {!Promise} A promise resolved when the certificate has been
+ * deleted successfully or rejected with a CertificatesError.
*/
deleteCertificate: function(id) {},
@@ -250,5 +262,6 @@ cr.define('settings', function() {
return {
CertificatesBrowserProxyImpl: CertificatesBrowserProxyImpl,
+ CertificateType: CertificateType,
};
});

Powered by Google App Engine
This is Rietveld 408576698