Commit 69c28dae authored by Abigail Klein's avatar Abigail Klein Committed by Chromium LUCI CQ

[Mac A11y] Remove the toolbar accessible name.

The name is redundant because the role is also toolbar. Keep the toolbar
as an AXNode so that it is a group for screen readers. This allows for
easier navigation around the chrome.

This is a partial revert of
https://chromium-review.googlesource.com/c/chromium/src/+/2588893.

name.

Bug: 1097760
Change-Id: I283d0d69b10db2e837575f6b2ce1ae3b7070a044
AX-Relnotes: Keep the toolbar as its own group but remove the redundant
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2605468Reviewed-by: default avatarElly Fong-Jones <ellyjones@chromium.org>
Commit-Queue: Abigail Klein <abigailbklein@google.com>
Cr-Commit-Position: refs/heads/master@{#840202}
parent 8ead0822
......@@ -5898,9 +5898,6 @@ the Bookmarks menu.">
<message name="IDS_ACCNAME_TAB_SEARCH" desc="The accessible name for the Tab Search bubble.">
Search tabs
</message>
<message name="IDS_ACCNAME_TOOLBAR" desc="The accessible name for the toolbar.">
Toolbar
</message>
<!-- Clipboard permission -->
<message name="IDS_ALLOWED_CLIPBOARD_TITLE" desc="Title of the info bubble shown when a site has been allowed access to read the contents of the system clipboard.">
......
4744ce7ac506516225561500cac78792e014b60e
\ No newline at end of file
......@@ -664,7 +664,6 @@ views::View* ToolbarView::GetDefaultFocusableChild() {
void ToolbarView::GetAccessibleNodeData(ui::AXNodeData* node_data) {
node_data->role = ax::mojom::Role::kToolbar;
node_data->SetName(l10n_util::GetStringUTF16(IDS_ACCNAME_TOOLBAR));
}
void ToolbarView::InitLayout() {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment