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

ui/views/accessibility/native_view_accessibility_win_unittest.cc

 // Copyright 2013 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 <oleacc.h>
 
 #include "base/win/scoped_bstr.h"
 #include "base/win/scoped_comptr.h"
 #include "base/win/scoped_variant.h"
 #include "third_party/iaccessible2/ia2_api_all.h"
 #include "ui/views/accessibility/native_view_accessibility.h"
 #include "ui/views/controls/textfield/textfield.h"
 #include "ui/views/test/views_test_base.h"
 
 using base::win::ScopedBstr;
 using base::win::ScopedComPtr;
 using base::win::ScopedVariant;
 
 namespace views {
 namespace test {
 
-class NativeViewAcccessibilityWinTest : public ViewsTestBase {
+class NativeViewAccessibilityWinTest : public ViewsTestBase {

[tapted, 2016/12/22 02:56:37]

this can be a separate patch I think :)

[Patti Lor, 2017/01/11 02:01:48]

Oops - thanks, I think I needed to edit stuff in here before but it wasn't
needed any more. Split off into https://codereview.chromium.org/2609443004/
which has already landed.

  public:
-  NativeViewAcccessibilityWinTest() {}
-  ~NativeViewAcccessibilityWinTest() override {}
+  NativeViewAccessibilityWinTest() {}
+  ~NativeViewAccessibilityWinTest() override {}
 
  protected:
   void GetIAccessible2InterfaceForView(View* view, IAccessible2_2** result) {
     ScopedComPtr<IAccessible> view_accessible(
         view->GetNativeViewAccessible());
     ScopedComPtr<IServiceProvider> service_provider;
     ASSERT_EQ(S_OK, view_accessible.QueryInterface(service_provider.Receive()));
     ASSERT_EQ(S_OK,
         service_provider->QueryService(IID_IAccessible2_2, result));
   }
 };
 
-TEST_F(NativeViewAcccessibilityWinTest, TextfieldAccessibility) {
+TEST_F(NativeViewAccessibilityWinTest, TextfieldAccessibility) {
   Widget widget;
   Widget::InitParams init_params =
       CreateParams(Widget::InitParams::TYPE_POPUP);
   init_params.ownership = Widget::InitParams::WIDGET_OWNS_NATIVE_WIDGET;
   widget.Init(init_params);
 
   View* content = new View;
   widget.SetContentsView(content);
 
   Textfield* textfield = new Textfield;
@@ skipping 25 matching lines @@
   ASSERT_EQ(S_OK, textfield_accessible->get_accValue(
       childid_self, value.Receive()));
   ASSERT_STREQ(L"Value", value);
 
   ScopedBstr new_value(L"New value");
   ASSERT_EQ(S_OK, textfield_accessible->put_accValue(childid_self, new_value));
 
   ASSERT_STREQ(L"New value", textfield->text().c_str());
 }
 
-TEST_F(NativeViewAcccessibilityWinTest, AuraOwnedWidgets) {
+TEST_F(NativeViewAccessibilityWinTest, AuraOwnedWidgets) {
   Widget widget;
   Widget::InitParams init_params =
       CreateParams(Widget::InitParams::TYPE_WINDOW);
   init_params.ownership = Widget::InitParams::WIDGET_OWNS_NATIVE_WIDGET;
   widget.Init(init_params);
 
   ScopedComPtr<IAccessible> root_view_accessible(
       widget.GetRootView()->GetNativeViewAccessible());
 
   LONG child_count = 0;
@@ skipping 42 matching lines @@
   ScopedComPtr<IDispatch> child_widget_parent_dispatch;
   ScopedComPtr<IAccessible> child_widget_parent_accessible;
   ASSERT_EQ(S_OK, child_widget_accessible->get_accParent(
       child_widget_parent_dispatch.Receive()));
   ASSERT_EQ(S_OK, child_widget_parent_dispatch.QueryInterface(
       child_widget_parent_accessible.Receive()));
   ASSERT_EQ(root_view_accessible.get(), child_widget_parent_accessible.get());
 }
 
 // Flaky on Windows: https://crbug.com/461837.
-TEST_F(NativeViewAcccessibilityWinTest, DISABLED_RetrieveAllAlerts) {
+TEST_F(NativeViewAccessibilityWinTest, DISABLED_RetrieveAllAlerts) {
   Widget widget;
   Widget::InitParams init_params =
       CreateParams(Widget::InitParams::TYPE_POPUP);
   init_params.ownership = Widget::InitParams::WIDGET_OWNS_NATIVE_WIDGET;
   widget.Init(init_params);
 
   View* content = new View;
   widget.SetContentsView(content);
 
   View* infobar = new View;
@@ skipping 45 matching lines @@
   delete infobar;
   ASSERT_EQ(S_OK, root_view_accessible->get_relationTargetsOfType(
       alerts_bstr, 0, &targets, &n_targets));
   ASSERT_EQ(1, n_targets);
   ASSERT_TRUE(infobar2_accessible.IsSameObject(targets[0]));
   CoTaskMemFree(targets);
 }
 
 }  // namespace test
 }  // namespace views