Skip to content

Commit

Permalink
Pass ShowAccessibilityOptionsInSystemTrayMenu policy to the a11y team
Browse files Browse the repository at this point in the history
As an effort to offload the policy team. Check details in the
https://crbug.com/1213081

Also added test cases for the policy

Bug: 1213081, 1213429, b:240372457
Change-Id: I1a813fb923372a6ea27591c9470ac41df18ee226
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3804569
Commit-Queue: Roman Sorokin <rsorokin@chromium.org>
Auto-Submit: Roman Sorokin <rsorokin@chromium.org>
Reviewed-by: Josiah Krutz <josiahk@google.com>
Commit-Queue: Josiah Krutz <josiahk@google.com>
Cr-Commit-Position: refs/heads/main@{#1030613}
  • Loading branch information
Roman Sorokin authored and Chromium LUCI CQ committed Aug 2, 2022
1 parent 21830df commit ee17a84
Show file tree
Hide file tree
Showing 2 changed files with 22 additions and 2 deletions.
22 changes: 21 additions & 1 deletion chrome/test/data/policy/policy_test_cases.json
Original file line number Diff line number Diff line change
Expand Up @@ -9333,12 +9333,32 @@
"chromeos_ash"
],
"policy_pref_mapping_tests": [
{
"note": "Default value (no policies set).",
"prefs": {
"settings.a11y.enable_menu": {
"default_value": false
}
}
},
{
"policies": {
"ShowAccessibilityOptionsInSystemTrayMenu": true
},
"prefs": {
"settings.a11y.enable_menu": {}
"settings.a11y.enable_menu": {
"value": true
}
}
},
{
"policies": {
"ShowAccessibilityOptionsInSystemTrayMenu": false
},
"prefs": {
"settings.a11y.enable_menu": {
"value": false
}
}
}
]
Expand Down
2 changes: 1 addition & 1 deletion components/policy/resources/policy_templates.json
Original file line number Diff line number Diff line change
Expand Up @@ -14968,7 +14968,7 @@
},
{
'name': 'ShowAccessibilityOptionsInSystemTrayMenu',
'owners': ['file://components/policy/resources/OWNERS', 'rsorokin@chromium.org'],
'owners': ['josiahk@google.com', 'file://ui/accessibility/OWNERS'],
'type': 'main',
'schema': { 'type': 'boolean' },
'supported_on': ['chrome_os:27-'],
Expand Down

0 comments on commit ee17a84

Please sign in to comment.