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

Unified Diff: chrome/test/data/extensions/api_test/settings_private/test.js

Issue 1163593005: Update chrome.settingsPrivate to support CrOS-only settings. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix issues preventing compilation on non-CrOS builds. Created 5 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/test/data/extensions/api_test/settings_private/test.js
diff --git a/chrome/test/data/extensions/api_test/settings_private/test.js b/chrome/test/data/extensions/api_test/settings_private/test.js
index 28ac7f644ab45427c87e58512b6766ee265e1206..e61166889368df9ef8a70d96697f4060fb3dc388 100644
--- a/chrome/test/data/extensions/api_test/settings_private/test.js
+++ b/chrome/test/data/extensions/api_test/settings_private/test.js
@@ -28,6 +28,16 @@ var availableTests = [
chrome.test.succeed();
});
},
+ function setPref_CrOSSetting() {
+ chrome.settingsPrivate.setPref(
+ 'cros.accounts.allowBWSI',
+ false,
+ kTestPageId,
+ function(success) {
+ callbackResult(success);
+ chrome.test.succeed();
+ });
+ },
function getPref() {
chrome.settingsPrivate.getPref(
kTestPrefName,
@@ -37,6 +47,15 @@ var availableTests = [
chrome.test.succeed();
});
},
+ function getPref_CrOSSetting() {
+ chrome.settingsPrivate.getPref(
+ 'cros.accounts.allowBWSI',
+ function(value) {
+ chrome.test.assertTrue(value !== null);
+ callbackResult(true);
+ chrome.test.succeed();
+ });
+ },
function getAllPrefs() {
chrome.settingsPrivate.getAllPrefs(
function(prefs) {
@@ -60,6 +79,21 @@ var availableTests = [
kTestPageId,
function() {});
},
+ function onPrefsChanged_CrOSSetting() {
+ chrome.settingsPrivate.onPrefsChanged.addListener(function(prefs) {
+ chrome.test.assertTrue(prefs.length > 0);
+ chrome.test.assertEq('cros.accounts.allowBWSI', prefs[0].key);
+ chrome.test.assertEq(false, prefs[0].value);
+ callbackResult(true);
+ chrome.test.succeed();
+ });
+
+ chrome.settingsPrivate.setPref(
+ 'cros.accounts.allowBWSI',
+ false,
+ kTestPageId,
+ function() {});
+ },
];
var testToRun = window.location.search.substring(1);

Powered by Google App Engine
This is Rietveld 408576698