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

Unified Diff: chrome/browser/ui/webui/options/certificate_manager_browsertest.js

Issue 2920253003: kiosk: Tighten cert manager UI (Closed)
Patch Set: Created 3 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/ui/webui/options/certificate_manager_browsertest.js
diff --git a/chrome/browser/ui/webui/options/certificate_manager_browsertest.js b/chrome/browser/ui/webui/options/certificate_manager_browsertest.js
index 0db91295ec92d822248e052975513f19b64042a0..4f30a7331c1b2da84581601ef211a04d0bb1a723 100644
--- a/chrome/browser/ui/webui/options/certificate_manager_browsertest.js
+++ b/chrome/browser/ui/webui/options/certificate_manager_browsertest.js
@@ -386,18 +386,22 @@ CertificateManagerStandaloneWebUITest.prototype = {
};
// Ensure that the standalone certificate manager page loads and displays the
-// ceertificates correctly.
+// certificates correctly.
TEST_F('CertificateManagerStandaloneWebUITest', 'testCertsDisplaying',
function() {
assertEquals(this.browsePreload, document.location.href);
// Click on the first folder and get the certificates.
- var caCertsTab = $('caCertsTab');
- caCertsTab.querySelector('div.tree-item').click();
- var certs = caCertsTab.querySelectorAll('div.tree-item div.tree-item');
+ var personalCertsTab = $('personalCertsTab');
+ personalCertsTab.querySelector('div.tree-item').click();
+ var certs = personalCertsTab.querySelectorAll('div.tree-item div.tree-item');
+
+ // There should be exactly one certificate displayed.
+ expectEquals(certs.length, 1);
- // There should be exactly three certificates displayed.
- expectEquals(certs.length, 3);
+ // No "Servers" and "Authorities" tab. See http://crbug.com/719907.
+ expectTrue($('server-certs-nav-tab').hidden);
+ expectTrue($('ca-certs-nav-tab').hidden);
});
GEN('#endif // defined(OS_CHROMEOS)');

Powered by Google App Engine
This is Rietveld 408576698