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

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

Issue 11189088: Add controlled setting indicator for default search engine configuration (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Comment addressed. 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/ui/webui/options/browser_options_handler.cc ('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 1fa55e8871732f8d8b3718f28ed494edffd35a93..354b61b2f5e4bb1cd8fd438bffd38d05c2d5d3ee 100644
--- a/chrome/test/data/policy/policy_test_cases.json
+++ b/chrome/test/data/policy/policy_test_cases.json
@@ -764,7 +764,12 @@
"test_policy": { "DefaultSearchProviderEnabled": false },
"settings_pages": [],
"pref_mappings": [
- { "pref": "default_search_provider.enabled" }
+ { "pref": "default_search_provider.enabled",
+ "indicator_tests": [
+ { "policy": { "DefaultSearchProviderEnabled": false } },
+ { "policy": { "DefaultSearchProviderEnabled": true, "DefaultSearchProviderSearchURL": "http://www.google.com/?q={searchTerms}" } }
+ ]
+ }
],
"note": "TODO(joaodasilva): Should be chrome://settings-frame. http://crbug.com/97749"
@@ -793,7 +798,12 @@
"test_policy": { "DefaultSearchProviderSearchURL": "http://www.google.com/?q={searchTerms}" },
"settings_pages": [],
"pref_mappings": [
- { "pref": "default_search_provider.search_url" }
+ { "pref": "default_search_provider.search_url",
+ "indicator_selector": "[setting=search-engine]",
+ "indicator_tests": [
+ { "policy": { "DefaultSearchProviderEnabled": true, "DefaultSearchProviderSearchURL": "http://www.google.com/?q={searchTerms}" } }
+ ]
+ }
],
"note": "TODO(joaodasilva): Should be chrome://settings-frame. http://crbug.com/97749"
« no previous file with comments | « chrome/browser/ui/webui/options/browser_options_handler.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698