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

Unified Diff: chrome/test/data/extensions/api_test/networking_private/chromeos/test.js

Issue 2891453002: Introduce networkingPrivate.getCertificateLists (Closed)
Patch Set: Rebase, fix histograms, remove from networking.onc 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/test/data/extensions/api_test/networking_private/chromeos/test.js
diff --git a/chrome/test/data/extensions/api_test/networking_private/chromeos/test.js b/chrome/test/data/extensions/api_test/networking_private/chromeos/test.js
index 388390dd53bf0cae6f36e707462163e369155eaa..400ac5af933af752ff74adb6495960a47f15fc0c 100644
--- a/chrome/test/data/extensions/api_test/networking_private/chromeos/test.js
+++ b/chrome/test/data/extensions/api_test/networking_private/chromeos/test.js
@@ -799,6 +799,12 @@ var availableTests = [
chrome.networkingPrivate.onDeviceStateListChanged.addListener(listener);
chrome.networkingPrivate.disableNetworkType('WiFi');
},
+ function onCertificateListsChangedEvent() {
+ chrome.test.listenOnce(
+ chrome.networkingPrivate.onCertificateListsChanged, function() {});
+ // The test delegate will send a change event when we request the certs.
tbarzic 2017/05/23 18:01:58 I'd go with something more explicit here, e.g. chr
stevenjb 2017/05/25 00:01:11 Ah, I knew there had to be a better way. I think I
+ chrome.networkingPrivate.getCertificateLists(callbackPass(function() {}));
+ },
function verifyDestination() {
chrome.networkingPrivate.verifyDestination(
verificationProperties,
@@ -971,6 +977,13 @@ var availableTests = [
'tetherName1', 50, 'tetherCarrier1', 75, true);
}));
},
+ function getCertificateLists() {
+ chrome.networkingPrivate.getCertificateLists(
+ callbackPass(function(certificateLists) {
+ assertEq(1, certificateLists.server_ca_certificates.length);
+ assertEq(0, certificateLists.user_certificates.length);
+ }));
+ },
];
chrome.test.getConfig(function(config) {

Powered by Google App Engine
This is Rietveld 408576698