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

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

Issue 2101243004: [MD settings] certificate manager closure compilation (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: clean Created 4 years, 6 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 204746853795e9fa0b3c2e9dd2289f1e0bcf97a7..ec6162930a7a4b4d0660edaa807b9835e856c066 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
@@ -9,28 +9,6 @@
/**
* @typedef {{
- * extractable: boolean,
- * id: string,
- * name: string,
- * policy: boolean,
- * readonly: boolean,
- * untrusted: boolean,
- * }}
- * @see chrome/browser/ui/webui/settings/certificates_handler.cc
- */
-var CertificateSubnode;
-
-/**
- * A data structure describing a certificate that is currently being imported,
- * therefore it has no ID yet, but it has a name. Used within JS only.
- * @typedef {{
- * name: string,
- * }}
- */
-var NewCertificateSubNode;
-
-/**
- * @typedef {{
* id: string,
* name: string,
* subnodes: !Array<!CertificateSubnode>
@@ -64,7 +42,7 @@ var CertificatesError;
* fail to be imported.
* @typedef {{
* title: string,
- * description: string
+ * description: string,
* certificateErrors: !Array<{name: string, error: string}>
* }}
* @see chrome/browser/ui/webui/settings/certificates_handler.cc
@@ -72,16 +50,6 @@ 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() {}
@@ -269,7 +237,7 @@ cr.define('settings', function() {
};
return {
+ CertificatesBrowserProxy: CertificatesBrowserProxy,
CertificatesBrowserProxyImpl: CertificatesBrowserProxyImpl,
- CertificateType: CertificateType,
};
});

Powered by Google App Engine
This is Rietveld 408576698