Name calculation should not include nameFrom:author descendants.

third_party/WebKit/Source/modules/accessibility/AXMenuList.h

 /*
  * Copyright (C) 2010 Apple Inc. All Rights Reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 25 matching lines @@
 class AXMenuList final : public AXLayoutObject {
   WTF_MAKE_NONCOPYABLE(AXMenuList);
 
  public:
   static AXMenuList* Create(LayoutMenuList* layout_object, AXObjectCacheImpl&);
 
   bool IsCollapsed() const override;
   AccessibilityExpanded IsExpanded() const final;
   bool Press() override;
   void ClearChildren() override;
-  bool NameFromContents() const override;
+  bool NameFromContents(const bool recursive) const override;

[dmazzoni, 2017/05/16 16:25:39]

I think const bool is redundant, I don't normally see
const for POD arguments because they're always pass-by-value.
Just use bool.

[aleventhal1, 2017/05/16 18:11:26]

Done.

 
   void DidUpdateActiveOption(int option_index);
   void DidShowPopup();
   void DidHidePopup();
 
  private:
   AXMenuList(LayoutMenuList*, AXObjectCacheImpl&);
 
   bool IsMenuList() const override { return true; }
   AccessibilityRole DetermineAccessibilityRole() final;
   bool CanSetFocusAttribute() const override;
 
   void AddChildren() override;
 };
 
 DEFINE_AX_OBJECT_TYPE_CASTS(AXMenuList, IsMenuList());
 
 }  // namespace blink
 
 #endif  // AXMenuList_h