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

Unified Diff: chrome/test/data/extensions/api_test/chromeos_info_private/extended/background.js

Issue 2686233006: Add managed device signal (Closed)
Patch Set: Minor comments change Created 3 years, 10 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/chromeos_info_private/extended/background.js
diff --git a/chrome/test/data/extensions/api_test/chromeos_info_private/extended/background.js b/chrome/test/data/extensions/api_test/chromeos_info_private/extended/background.js
index 9297957c946466f62b06adb07c8f5f7e0bd62b36..fbc121a53f86955bff82097d674384b908c12097 100644
--- a/chrome/test/data/extensions/api_test/chromeos_info_private/extended/background.js
+++ b/chrome/test/data/extensions/api_test/chromeos_info_private/extended/background.js
@@ -9,18 +9,21 @@ chrome.app.runtime.onLaunched.addListener(function() {
chrome.test.fail("Missing test name.");
return;
}
- chrome.chromeosInfoPrivate.get(['sessionType', 'playStoreStatus'],
- function (values) {
- if (testName == 'kiosk') {
- chrome.test.assertEq('kiosk', values['sessionType']);
- } else if (testName == 'arc not-available') {
- chrome.test.assertEq('not available', values['playStoreStatus']);
- } else if (testName == 'arc available') {
- chrome.test.assertEq('available', values['playStoreStatus']);
- } else if (testName == 'arc enabled') {
- chrome.test.assertEq('enabled', values['playStoreStatus']);
- }
- chrome.test.succeed();
- });
+ chrome.chromeosInfoPrivate.get(
+ ['sessionType', 'playStoreStatus', 'managedDeviceStatus'],
+ function(values) {
tbarzic 2017/02/22 02:18:35 You can use chrome.test.callbackPass(function(valu
Wenzhao (Colin) Zang 2017/02/22 03:11:07 Done.
+ if (testName == 'kiosk') {
+ chrome.test.assertEq('kiosk', values['sessionType']);
+ } else if (testName == 'arc not-available') {
tbarzic 2017/02/22 02:18:35 nit: maybe switch this to switch {case:}
Wenzhao (Colin) Zang 2017/02/22 03:11:07 Done.
+ chrome.test.assertEq('not available', values['playStoreStatus']);
+ } else if (testName == 'arc available') {
+ chrome.test.assertEq('available', values['playStoreStatus']);
+ } else if (testName == 'arc enabled') {
+ chrome.test.assertEq('enabled', values['playStoreStatus']);
+ } else if (testName == 'managed') {
+ chrome.test.assertEq('managed', values['managedDeviceStatus']);
+ }
+ chrome.test.succeed();
+ });
});
});

Powered by Google App Engine
This is Rietveld 408576698