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

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

Issue 1283613005: Add networkingPrivate.getThirdPartyVpnProviderList (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@issue_515987_cr_network_list
Patch Set: Rebase Created 5 years, 4 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 5127715db9e46c65bda0009933758717ac0144f8..89eee73261ce729cc66f03436026f834c54acc1e 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
@@ -839,7 +839,42 @@ var availableTests = [
}));
}));
})));
- }
+ },
+ function getThirdPartyVPNProviders() {
+ var kVpnTestExtensionId = 'dnlobbijiphininmjcaipninhcdnknni';
+ var kVpmTestExtensionName = 'Basic tests';
+ chrome.networkingPrivate.getThirdPartyVPNProviders(
+ callbackPass(function(result) {
+ assertEq([{
+ ExtensionID: kVpnTestExtensionId,
+ Name: kVpmTestExtensionName,
+ }
+ ], result);
+ }));
+ },
+ function onThirdPartyVPNProvidersChangedEvent() {
+ var kVpnTestExtensionId = 'dnlobbijiphininmjcaipninhcdnknni';
+ var kVpmTestExtensionName = 'Basic tests';
+ var done = chrome.test.callbackAdded();
pneubeck (no reviews) 2015/08/13 12:58:36 nit: you could still use the callbackPass wrapper
+ var listener = function(providers) {
+ chrome.networkingPrivate.onThirdPartyVPNProvidersChanged.removeListener(
+ listener);
+ assertEq([{
+ ExtensionID: kVpnTestExtensionId,
+ Name: kVpmTestExtensionName,
+ }], providers);
+ done();
+ };
+ chrome.networkingPrivate.onThirdPartyVPNProvidersChanged.addListener(
+ listener);
+ chrome.networkingPrivate.getThirdPartyVPNProviders(function(result) {
+ // Ensure no initial providers.
+ assertEq([], result);
+ // Tell Chrome to install a third party VPN extension which should trigger
+ // |listener|.
+ chrome.test.sendMessage('installThirdPartyVpn');
+ });
+ },
];
var testToRun = window.location.search.substring(1);

Powered by Google App Engine
This is Rietveld 408576698