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

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

Issue 2410113002: Add migration code for the EnabledPlugins and DisabledPlugins policies. (Closed)
Patch Set: Remove case insensitivity. Created 4 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/policy/configuration_policy_handler_list_factory.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 a886d4aa434673ad5d0221c40a6f95ac7b93988d..ac11a31063872ee798dd3d2a58ccd9ffc29259b1 100644
--- a/chrome/test/data/policy/policy_test_cases.json
+++ b/chrome/test/data/policy/policy_test_cases.json
@@ -367,30 +367,67 @@
]
},
- "DisabledPlugins": {
+ "DisabledPlugins.0": {
+ "note": "This policy is deprecated. This test tests its migration path.",
"os": ["win", "linux", "mac", "chromeos"],
- "test_policy": { "DisabledPlugins": ["Flash"] },
+ "test_policy": { "DisabledPlugins": ["*Flash*"] },
"pref_mappings": [
- { "pref": "plugins.plugins_disabled" }
+ { "pref": "profile.managed_default_content_settings.plugins",
+ "indicator_selector": "[content-setting=plugins]",
+ "indicator_tests": [
+ { "policy": { "DisabledPlugins": ["*Flash*"] },
+ "value": "block"}
+ ]
+ }
]
},
- "EnabledPlugins": {
+ "DisabledPlugins.1": {
+ "note": "This policy is deprecated. This test tests its migration path.",
"os": ["win", "linux", "mac", "chromeos"],
- "test_policy": { "EnabledPlugins": ["Flash"] },
+ "test_policy": { "DisabledPlugins": ["*PDF*"] },
"pref_mappings": [
- { "pref": "plugins.plugins_enabled" }
+ { "pref": "plugins.always_open_pdf_externally",
+ "indicator_tests": [
+ { "policy": { "DisabledPlugins": ["*PDF*"] }}
+ ]
+ }
]
},
- "DisabledPluginsExceptions": {
+ "EnabledPlugins.0": {
+ "note": "This policy is deprecated. This test tests its migration path.",
+ "os": ["win", "linux", "mac", "chromeos"],
+ "test_policy": { "EnabledPlugins": ["*Flash*"] },
+ "pref_mappings": [
+ { "pref": "profile.managed_default_content_settings.plugins",
+ "indicator_selector": "[content-setting=plugins]",
+ "indicator_tests": [
+ { "policy": { "EnabledPlugins": ["*Flash*"] },
+ "value": "allow"}
+ ]
+ }
+ ]
+ },
+
+ "EnabledPlugins.1": {
+ "note": "This policy is deprecated. This test tests its migration path.",
"os": ["win", "linux", "mac", "chromeos"],
- "test_policy": { "DisabledPluginsExceptions": ["Flash"] },
+ "test_policy": { "EnabledPlugins": ["*PDF*"] },
"pref_mappings": [
- { "pref": "plugins.plugins_disabled_exceptions" }
+ { "pref": "plugins.always_open_pdf_externally",
+ "indicator_tests": [
+ { "policy": { "EnabledPlugins": ["*PDF*"] }}
+ ]
+ }
]
},
+ "DisabledPluginsExceptions": {
+ "note": "This policy is deprecated it doesn't map to a pref value anymore.",
+ "os": ["win", "linux", "mac", "chromeos"]
+ },
+
"AlwaysOpenPdfExternally": {
"os": ["win", "linux", "mac"],
"test_policy": { "AlwaysOpenPdfExternally": true },
« no previous file with comments | « chrome/browser/policy/configuration_policy_handler_list_factory.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698