Fix accelerator handling for in-menu buttons in the app menu.

ui/views/view.h

 // Copyright (c) 2012 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.
 
 #ifndef UI_VIEWS_VIEW_H_
 #define UI_VIEWS_VIEW_H_
 
 #include <stddef.h>
 
 #include <algorithm>
@@ skipping 1413 matching lines @@
 
   // Starts a drag and drop operation originating from this view. This invokes
   // WriteDragData to write the data and GetDragOperations to determine the
   // supported drag operations. When done, OnDragDone is invoked. |press_pt| is
   // in the view's coordinate system.
   // Returns true if a drag was started.
   bool DoDrag(const ui::LocatedEvent& event,
               const gfx::Point& press_pt,
               ui::DragDropTypes::DragEventSource source);
 
+  // Returns true if this is not a top level widget. Virtual for tests.
+  virtual bool IsChildWidget() const;
+  // Returns true if the focus is not in a top level widget. Virtual for tests.
+  virtual bool FocusInChildWidget() const;

[sadrul, 2016/01/28 08:11:00]

These should not be virtual methods. It should be easy to set-up the hierarchy
in tests.

[meacer, 2016/01/29 00:37:36]

Done.

+
   //////////////////////////////////////////////////////////////////////////////
 
   // Creation and lifetime -----------------------------------------------------
 
   // False if this View is owned by its parent - i.e. it will be deleted by its
   // parent during its parents destruction. False is the default.
   bool owned_by_client_;
 
   // Attributes ----------------------------------------------------------------
 
@@ skipping 130 matching lines @@
   // Belongs to this view, but it's reference-counted on some platforms
   // so we can't use a scoped_ptr. It's dereferenced in the destructor.
   NativeViewAccessibility* native_view_accessibility_;
 
   DISALLOW_COPY_AND_ASSIGN(View);
 };
 
 }  // namespace views
 
 #endif  // UI_VIEWS_VIEW_H_