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

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

Issue 2410113002: Add migration code for the EnabledPlugins and DisabledPlugins policies. (Closed)
Patch Set: Invert the matching. 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
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..51b5e30f084fc4ba2781182c7290d9c146e4288d 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"] },
"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"}
+ ]
+ }
+ ]
+ },
+
+ "DisabledPlugins.1": {
+ "note": "This policy is deprecated. This test tests its migration path.",
+ "os": ["win", "linux", "mac", "chromeos"],
+ "test_policy": { "DisabledPlugins": ["Some PDF Plugin"] },
+ "pref_mappings": [
+ { "pref": "plugins.always_open_pdf_externally",
+ "indicator_tests": [
+ { "policy": { "DisabledPlugins": ["Some PDF Plugin"] }}
+ ]
+ }
]
},
- "EnabledPlugins": {
+ "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": "plugins.plugins_enabled" }
+ { "pref": "profile.managed_default_content_settings.plugins",
+ "indicator_selector": "[content-setting=plugins]",
+ "indicator_tests": [
+ { "policy": { "EnabledPlugins": ["Flash"] },
+ "value": "allow"}
+ ]
+ }
]
},
- "DisabledPluginsExceptions": {
+ "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": ["Some PDF Plugin"] },
"pref_mappings": [
- { "pref": "plugins.plugins_disabled_exceptions" }
+ { "pref": "plugins.always_open_pdf_externally",
+ "indicator_tests": [
+ { "policy": { "EnabledPlugins": ["Some PDF Plugin"] }}
+ ]
+ }
]
},
+ "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 },

Powered by Google App Engine
This is Rietveld 408576698