MD Bookmarks: Initial screenreader accessibility improvements

chrome/browser/resources/md_bookmarks/toolbar.html

Index: chrome/browser/resources/md_bookmarks/toolbar.html
diff --git a/chrome/browser/resources/md_bookmarks/toolbar.html b/chrome/browser/resources/md_bookmarks/toolbar.html
index 529b3bdbd93a5fc8f15fc499edd40cb0fc223b26..9f077ac3d729302dfa33d5c71588e474d587fb2c 100644
--- a/chrome/browser/resources/md_bookmarks/toolbar.html
+++ b/chrome/browser/resources/md_bookmarks/toolbar.html
@@ -55,6 +55,7 @@
       <button is="paper-icon-button-light"
           id="menuButton"
           class="more-actions more-vert-button"
+          title="$i18n{organizeButtonTitle}"
           on-tap="onMenuButtonOpenTap_">
         <div></div>
         <div></div>
@@ -62,26 +63,38 @@
       </button>
     </cr-toolbar>
     <template is="cr-lazy-render" id="dropdown">
-      <dialog is="cr-action-menu">
-        <button class="dropdown-item" on-tap="onSortTap_"
-            disabled="[[!canChangeList_]]">
+      <dialog is="cr-action-menu" role="menu">
+        <button class="dropdown-item"
+            on-tap="onSortTap_"
+            disabled="[[!canChangeList_]]"
+            role="menuitem">

[calamity, 2017/06/30 06:05:18]

Still no way around the top level menu issue?

[tsergeant, 2017/07/03 04:07:19]

Do you mean the one where VoiceOver was getting caught up on the item wrapper
div?

The behavior seems more reasonable on ChromeVox than Voiceover. We can check
with a11y review if there's anything better we can do after this lands.

           $i18n{menuSort}
         </button>
         <hr>
-        <button id="addBookmarkButton" class="dropdown-item"
-            on-tap="onAddBookmarkTap_" disabled="[[!canChangeList_]]">
+        <button id="addBookmarkButton"
+            class="dropdown-item"
+            on-tap="onAddBookmarkTap_"
+            disabled="[[!canChangeList_]]"
+            role="menuitem">
           $i18n{menuAddBookmark}
         </button>
-        <button class="dropdown-item" on-tap="onAddFolderTap_"
-            disabled="[[!canChangeList_]]">
+        <button class="dropdown-item"
+            on-tap="onAddFolderTap_"
+            disabled="[[!canChangeList_]]"
+            role="menuitem">
           $i18n{menuAddFolder}
         </button>
         <hr>
-        <button id="importBookmarkButton" class="dropdown-item"
-            on-tap="onImportTap_" disabled="[[!globalCanEdit_]]">
+        <button id="importBookmarkButton"
+            class="dropdown-item"
+            on-tap="onImportTap_"
+            disabled="[[!globalCanEdit_]]"
+            role="menuitem">
           $i18n{menuImport}
         </button>
-        <button class="dropdown-item" on-tap="onExportTap_">
+        <button class="dropdown-item"
+            on-tap="onExportTap_"
+            role="menuitem">
           $i18n{menuExport}
         </button>
       </dialog>