Forward BrowserAccessibilityWin::get_accKeyboardShortcut to the platform node

content/renderer/accessibility/blink_ax_tree_source.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/renderer/accessibility/blink_ax_tree_source.h"
 
 #include <stddef.h>
 
 #include <set>
 
@@ skipping 543 matching lines @@
     if (src.HasComputedStyle()) {
       dst->AddStringAttribute(ui::AX_ATTR_DISPLAY,
                               src.ComputedStyleDisplay().Utf8());
     }
 
     if (src.Language().length()) {
       if (parent.IsNull() || parent.Language() != src.Language())
         dst->AddStringAttribute(ui::AX_ATTR_LANGUAGE, src.Language().Utf8());
     }
 
     if (src.KeyboardShortcut().length()) {

[dmazzoni, 2017/06/06 04:44:17]

Note that AX_ATTR_KEY_SHORTCUTS may already be set above, in
GetSparseAXAttributes.

Let's skip this one if that one already set that attribute. Change this to:

if (src.KeyboardShortcut().length() &&
!dst->HasStringAttribute(ui::AX_ATTR_KEY_SHORTCUTS))

-      dst->AddStringAttribute(ui::AX_ATTR_SHORTCUT,
+      dst->AddStringAttribute(ui::AX_ATTR_KEY_SHORTCUTS,
                               src.KeyboardShortcut().Utf8());
     }
 
     if (!src.NextOnLine().IsDetached()) {
       dst->AddIntAttribute(ui::AX_ATTR_NEXT_ON_LINE_ID,
                            src.NextOnLine().AxID());
     }
 
     if (!src.PreviousOnLine().IsDetached()) {
       dst->AddIntAttribute(ui::AX_ATTR_PREVIOUS_ON_LINE_ID,
@@ skipping 312 matching lines @@
     return WebAXObject();
 
   WebDocument document = render_frame_->GetWebFrame()->GetDocument();
   if (!document.IsNull())
     return document.AccessibilityObject();
 
   return WebAXObject();
 }
 
 }  // namespace content