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

Unified Diff: chrome/test/data/policy/policy_test_cases.json

Issue 11198065: Add fourth batch of controlled setting indicators (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Removed two obsoleted test cases. Created 8 years, 2 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
« no previous file with comments | « chrome/browser/resources/options/startup_section.html ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/data/policy/policy_test_cases.json
diff --git a/chrome/test/data/policy/policy_test_cases.json b/chrome/test/data/policy/policy_test_cases.json
index 1897ddada4a9fe85455907724c4108dfcdb2721b..6b9fe28fee977ced4548a8fe5b799589fb749119 100644
--- a/chrome/test/data/policy/policy_test_cases.json
+++ b/chrome/test/data/policy/policy_test_cases.json
@@ -155,8 +155,7 @@
"indicator_selector": "[content-setting=javascript]",
"indicator_tests": [
{ "policy": { "JavascriptEnabled": false },
- "value": "block"},
- { "policy": { "JavascriptEnabled": true } }
+ "value": "block"}
]
}
]
@@ -447,7 +446,6 @@
{ "pref": "profile.managed_default_content_settings.cookies",
"indicator_selector": "[content-setting=cookies]",
"indicator_tests": [
- { "policy": { "ClearSiteDataOnExit": false } },
{ "policy": { "ClearSiteDataOnExit": true },
"value": "session"}
]
@@ -687,10 +685,15 @@
"ShowHomeButton": {
"os": ["win", "linux", "mac", "chromeos"],
+ "can_be_recommended": true,
"test_policy": { "ShowHomeButton": true },
"settings_pages": ["chrome://settings-frame"],
"pref_mappings": [
- { "pref": "browser.show_home_button" }
+ { "pref": "browser.show_home_button",
+ "indicator_tests": [
+ { "policy": { "ShowHomeButton": true } }
+ ]
+ }
]
},
@@ -705,19 +708,34 @@
"RestoreOnStartup": {
"os": ["win", "linux", "mac", "chromeos"],
+ "can_be_recommended": true,
"test_policy": { "RestoreOnStartup": 4 },
"settings_pages": ["chrome://settings-frame"],
"pref_mappings": [
- { "pref": "session.restore_on_startup" }
+ { "pref": "session.restore_on_startup",
+ "indicator_tests": [
+ { "policy": { "RestoreOnStartup": 1 },
+ "value": "1"},
+ { "policy": { "RestoreOnStartup": 4 },
+ "value": "4"},
+ { "policy": { "RestoreOnStartup": 5 },
+ "value": "5"}
+ ]
+ }
]
},
"RestoreOnStartupURLs": {
"os": ["win", "linux", "mac", "chromeos"],
+ "can_be_recommended": true,
"test_policy": { "RestoreOnStartupURLs": ["chromium.org"] },
"settings_pages": [],
"pref_mappings": [
- { "pref": "session.urls_to_restore_on_startup" }
+ { "pref": "session.urls_to_restore_on_startup",
+ "indicator_tests": [
+ { "policy": { "RestoreOnStartupURLs": ["chromium.org"] } }
+ ]
+ }
],
"note": "TODO(joaodasilva): Should be chrome://settings-frame. http://crbug.com/97749"
« no previous file with comments | « chrome/browser/resources/options/startup_section.html ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698