a11y: Exclude children of nested keyboard accessible controls from a11y tree.

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 761 matching lines @@
   // Returns the view that should be selected next when pressing Shift-Tab.
   View* GetPreviousFocusableView();
 
   // Sets the component that should be selected next when pressing Tab, and
   // makes the current view the precedent view of the specified one.
   // Note that by default views are linked in the order they have been added to
   // their container. Use this method if you want to modify the order.
   // IMPORTANT NOTE: loops in the focus hierarchy are not supported.
   void SetNextFocusableView(View* view);
 
-  // Sets |focus_behavior| and advances focus if necessary.
+  // Gets/sets |focus_behavior|. SetFocusBehavior() advances focus if necessary.
+  FocusBehavior focus_behavior() const { return focus_behavior_; }

[tapted, 2016/11/29 03:12:49]

I think this is OK. (although it raises the question of whether
`IsAccessibilityFocusable`, or the FocusBehaviour enum values deserve better
names..). OWNERS may have some other suggestions

[Patti Lor, 2016/11/29 23:26:18]

Acknowledged.

   void SetFocusBehavior(FocusBehavior focus_behavior);
 
   // Returns true if this view is focusable, |enabled_| and drawn.
   bool IsFocusable() const;
 
   // Return whether this view is focusable when the user requires full keyboard
   // access, even though it may not be normally focusable.
   bool IsAccessibilityFocusable() const;
 
   // Convenience method to retrieve the FocusManager associated with the
@@ skipping 354 matching lines @@
   // with an associated view. Widget::ReorderNativeViews() may reorder layers
   // below layers owned by a view.
   virtual void ReorderChildLayers(ui::Layer* parent_layer);
 
   // Input ---------------------------------------------------------------------
 
   virtual DragInfo* GetDragInfo();
 
   // Focus ---------------------------------------------------------------------
 
-  // Returns last set focus behavior.
-  FocusBehavior focus_behavior() const { return focus_behavior_; }
-
   // Override to be notified when focus has changed either to or from this View.
   virtual void OnFocus();
   virtual void OnBlur();
 
   // Handle view focus/blur events for this view.
   void Focus();
   void Blur();
 
   // System events -------------------------------------------------------------
 
@@ skipping 413 matching lines @@
   // Observers -------------------------------------------------------------
 
   base::ObserverList<ViewObserver> observers_;
 
   DISALLOW_COPY_AND_ASSIGN(View);
 };
 
 }  // namespace views
 
 #endif  // UI_VIEWS_VIEW_H_