Forward BrowserAccessibility::get_accSelection to AXPlatformNode.

ui/accessibility/platform/ax_platform_node_win_unittest.cc

 // Copyright 2015 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 "ui/accessibility/platform/ax_platform_node.h"
 
 #include <atlbase.h>
 #include <atlcom.h>
 #include <oleacc.h>
 
@@ skipping 56 matching lines @@
   void Init(const AXNodeData& node1,
             const AXNodeData& node2,
             const AXNodeData& node3) {
     AXTreeUpdate update;
     update.root_id = node1.id;
     update.nodes.push_back(node1);
     update.nodes.push_back(node2);
     update.nodes.push_back(node3);
     Init(update);
   }
+  void Init(const AXNodeData& node1,
+            const AXNodeData& node2,
+            const AXNodeData& node3,
+            const AXNodeData& node4) {
+    AXTreeUpdate update;
+    update.root_id = node1.id;
+    update.nodes.push_back(node1);
+    update.nodes.push_back(node2);
+    update.nodes.push_back(node3);
+    update.nodes.push_back(node4);
+    Init(update);
+  }
 
  protected:
   AXNode* GetRootNode() {
     return tree_->root();
   }
 
   ScopedComPtr<IAccessible> IAccessibleFromNode(AXNode* node) {
     TestAXNodeWrapper* wrapper =
         TestAXNodeWrapper::GetOrCreate(tree_.get(), node);
     if (!wrapper)
@@ skipping 99 matching lines @@
   EXPECT_EQ(S_OK, root_obj->get_accKeyboardShortcut(SELF, shortcut.Receive()));
   EXPECT_STREQ(L"Shortcut", shortcut);
 
   EXPECT_EQ(E_INVALIDARG, root_obj->get_accKeyboardShortcut(SELF, nullptr));
   ScopedVariant bad_id(999);
   ScopedBstr k2;
   EXPECT_EQ(E_INVALIDARG,
             root_obj->get_accKeyboardShortcut(bad_id, k2.Receive()));
 }
 
+TEST_F(AXPlatformNodeWinTest, TestIAccessibleSelectionNotListBox) {
+  // We only support AX_ROLE_LIST_BOX as this point, so, this should return
+  // not implemented. We're choosing AX_ROLE_ALERT, but it could be anything
+  // but AX_ROLE_LIST_BOX_OPTION.
+
+  AXNodeData not_supported;
+  not_supported.id = 0;
+  not_supported.role = AX_ROLE_ALERT;
+
+  Init(not_supported);
+  ScopedComPtr<IAccessible> root_obj(GetRootIAccessible());
+
+  ScopedVariant selection;
+  EXPECT_EQ(E_NOTIMPL, root_obj->get_accSelection(selection.Receive()));
+}
+
+TEST_F(AXPlatformNodeWinTest, TestIAccessibleSelectionNothingSelected) {
+  // We're going to set up a AX_ROLE_LIST_BOX_OPTION with 2 options with
+  // nothing selected
+  AXNodeData list;
+  list.id = 0;
+  list.role = AX_ROLE_LIST_BOX;
+
+  list.child_ids.push_back(2);
+  list.child_ids.push_back(3);
+
+  AXNodeData list_item_2;
+  list_item_2.id = 2;
+  list_item_2.role = AX_ROLE_LIST_BOX_OPTION;
+  list_item_2.AddStringAttribute(AX_ATTR_NAME, "Name2");
+
+  AXNodeData list_item_3;
+  list_item_3.id = 3;
+  list_item_3.role = AX_ROLE_LIST_BOX_OPTION;
+  list_item_3.AddStringAttribute(AX_ATTR_NAME, "Name3");
+
+  // Nothing is selected.  This should return S_OK and the selection should
+  // be VT_EMPTY.
+
+  Init(list, list_item_2, list_item_3);
+  ScopedComPtr<IAccessible> root_obj(GetRootIAccessible());
+
+  ScopedVariant selection;
+  EXPECT_EQ(S_OK, root_obj->get_accSelection(selection.Receive()));
+  EXPECT_EQ(VT_EMPTY, selection.type());
+}
+
+TEST_F(AXPlatformNodeWinTest, TestIAccessibleSelectionOneSelected) {
+  // We're going to set up a AX_ROLE_LIST_BOX_OPTION with 2 options with
+  // one selected.
+  AXNodeData list;
+  list.id = 0;
+  list.role = AX_ROLE_LIST_BOX;
+
+  list.child_ids.push_back(2);
+  list.child_ids.push_back(3);
+
+  AXNodeData list_item_2;
+  list_item_2.id = 2;
+  list_item_2.role = AX_ROLE_LIST_BOX_OPTION;
+  list_item_2.state = 1 << ui::AX_STATE_SELECTED;
+  list_item_2.AddStringAttribute(AX_ATTR_NAME, "Name2");
+
+  AXNodeData list_item_3;
+  list_item_3.id = 3;
+  list_item_3.role = AX_ROLE_LIST_BOX_OPTION;
+  list_item_3.AddStringAttribute(AX_ATTR_NAME, "Name3");
+
+  Init(list, list_item_2, list_item_3);
+
+  ScopedComPtr<IAccessible> root_obj(GetRootIAccessible());
+
+  ScopedVariant selection;
+  EXPECT_EQ(S_OK, root_obj->get_accSelection(selection.Receive()));
+  ASSERT_NE(nullptr, selection.ptr());
+
+  // We got something back, make sure that it has the correct name.
+  base::win::ScopedComPtr<IAccessible> accessible;
+  HRESULT hr =
+      V_DISPATCH(selection.ptr())->QueryInterface(IID_PPV_ARGS(&accessible));
+  EXPECT_EQ(S_OK, hr);
+  ScopedBstr name;
+  EXPECT_EQ(S_OK, accessible->get_accName(SELF, name.Receive()));
+  EXPECT_STREQ(L"Name2", name);
+}
+
+TEST_F(AXPlatformNodeWinTest, TestIAccessibleSelectionMultipleSelected) {
+  // We're going to set up a AX_ROLE_LIST_BOX_OPTION with 3 options with
+  // two selected.
+  AXNodeData list;
+  list.id = 0;
+  list.role = AX_ROLE_LIST_BOX;
+
+  list.child_ids.push_back(2);
+  list.child_ids.push_back(3);
+  list.child_ids.push_back(4);
+
+  AXNodeData list_item_2;
+  list_item_2.id = 2;
+  list_item_2.role = AX_ROLE_LIST_BOX_OPTION;
+  list_item_2.state = 1 << ui::AX_STATE_SELECTED;
+  list_item_2.AddStringAttribute(AX_ATTR_NAME, "Name2");
+
+  AXNodeData list_item_3;
+  list_item_3.id = 3;
+  list_item_3.role = AX_ROLE_LIST_BOX_OPTION;
+  list_item_3.state = 1 << ui::AX_STATE_SELECTED;
+  list_item_3.AddStringAttribute(AX_ATTR_NAME, "Name3");
+
+  AXNodeData list_item_4;
+  list_item_4.id = 4;
+  list_item_4.role = AX_ROLE_LIST_BOX_OPTION;
+  list_item_4.AddStringAttribute(AX_ATTR_NAME, "Name4");
+  Init(list, list_item_2, list_item_3, list_item_4);
+
+  ScopedComPtr<IAccessible> root_obj(GetRootIAccessible());
+
+  ScopedVariant selection;
+  EXPECT_EQ(S_OK, root_obj->get_accSelection(selection.Receive()));
+  ASSERT_NE(nullptr, selection.ptr());
+
+  // We got something back, make sure that it has the corrent name.
+  base::win::ScopedComPtr<IEnumVARIANT> accessibles;
+  HRESULT hr =
+      V_DISPATCH(selection.ptr())->QueryInterface(IID_PPV_ARGS(&accessibles));
+  EXPECT_EQ(S_OK, hr);
+  ULONG ignore;
+
+  // Check out the first selected item.
+  {
+    ScopedVariant item;
+    hr = accessibles->Next(1, item.Receive(), &ignore);
+    EXPECT_EQ(S_OK, hr);
+
+    base::win::ScopedComPtr<IAccessible> accessible;

[dmazzoni, 2017/06/26 16:58:31]

Not necessary for this change, but I wonder if there
could be some helper functions at the top of this
file to remove some boilerplate from some of these
checks.

It's not immediately obvious what it'd be. Let's
think about it.

[dougt, 2017/06/26 22:19:11]

Agreed. At the very least, if we continue adding similar checks, we should
factor out the "Test that this IAccessible is the expected one".

+    HRESULT hr =
+        V_DISPATCH(item.ptr())->QueryInterface(IID_PPV_ARGS(&accessible));
+    EXPECT_EQ(S_OK, hr);
+    ScopedBstr name;
+    EXPECT_EQ(S_OK, accessible->get_accName(SELF, name.Receive()));
+    EXPECT_STREQ(L"Name2", name);
+  }
+
+  // and the second selected element.
+  {
+    ScopedVariant item;
+    hr = accessibles->Next(1, item.Receive(), &ignore);
+    EXPECT_EQ(S_OK, hr);
+
+    base::win::ScopedComPtr<IAccessible> accessible;
+    HRESULT hr =
+        V_DISPATCH(item.ptr())->QueryInterface(IID_PPV_ARGS(&accessible));
+    EXPECT_EQ(S_OK, hr);
+    ScopedBstr name;
+    EXPECT_EQ(S_OK, accessible->get_accName(SELF, name.Receive()));
+    EXPECT_STREQ(L"Name3", name);
+  }
+
+  // There shouldn't be any more selected.
+  {
+    ScopedVariant item;
+    hr = accessibles->Next(1, item.Receive(), &ignore);
+    EXPECT_EQ(S_FALSE, hr);
+  }
+}
+
 TEST_F(AXPlatformNodeWinTest, TestIAccessibleRole) {
   AXNodeData root;
   root.id = 1;
   root.child_ids.push_back(2);
 
   AXNodeData child;
   child.id = 2;
 
   Init(root, child);
   AXNode* child_node = GetRootNode()->children()[0];
@@ skipping 307 matching lines @@
   EXPECT_HRESULT_SUCCEEDED(text_field->setSelection(0, 1, 0));
   EXPECT_HRESULT_SUCCEEDED(text_field->setSelection(0, 2, 2));
   EXPECT_HRESULT_SUCCEEDED(text_field->setSelection(0, IA2_TEXT_OFFSET_CARET,
                                                     IA2_TEXT_OFFSET_LENGTH));
 
   EXPECT_HRESULT_FAILED(text_field->setSelection(1, 0, 0));
   EXPECT_HRESULT_FAILED(text_field->setSelection(0, 0, 5));
 }
 
 }  // namespace ui