Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(695)

Unified Diff: ui/accessibility/ax_tree_unittest.cc

Issue 2694903010: AX checked state changes (Closed)
Patch Set: git cl try Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ui/accessibility/ax_tree_unittest.cc
diff --git a/ui/accessibility/ax_tree_unittest.cc b/ui/accessibility/ax_tree_unittest.cc
index fdf41f9b1095de22ee7f229292be1f6e89afccc2..7c13c19caca1a0fbdac1735b19a1c9580eb62d21 100644
--- a/ui/accessibility/ax_tree_unittest.cc
+++ b/ui/accessibility/ax_tree_unittest.cc
@@ -627,7 +627,8 @@ TEST(AXTreeTest, RoleAndStateChangeCallbacks) {
initial_state.nodes[0].id = 1;
initial_state.nodes[0].role = AX_ROLE_BUTTON;
initial_state.nodes[0].state = 0;
- initial_state.nodes[0].AddStateFlag(AX_STATE_CHECKED);
+ initial_state.nodes[0].AddIntAttribute(ui::AX_ATTR_CHECKED_STATE,
+ ui::AX_CHECKED_STATE_TRUE);
initial_state.nodes[0].AddStateFlag(AX_STATE_FOCUSABLE);
AXTree tree(initial_state);
@@ -641,6 +642,8 @@ TEST(AXTreeTest, RoleAndStateChangeCallbacks) {
update.nodes[0].id = 1;
update.nodes[0].role = AX_ROLE_CHECK_BOX;
update.nodes[0].state = 0;
+ update.nodes[0].AddIntAttribute(ui::AX_ATTR_CHECKED_STATE,
+ ui::AX_CHECKED_STATE_FALSE);
update.nodes[0].AddStateFlag(AX_STATE_FOCUSABLE);
update.nodes[0].AddStateFlag(AX_STATE_VISITED);
EXPECT_TRUE(tree.Unserialize(update));
@@ -649,8 +652,8 @@ TEST(AXTreeTest, RoleAndStateChangeCallbacks) {
fake_delegate.attribute_change_log();
ASSERT_EQ(3U, change_log.size());
EXPECT_EQ("Role changed from button to checkBox", change_log[0]);
- EXPECT_EQ("checked changed to false", change_log[1]);
- EXPECT_EQ("visited changed to true", change_log[2]);
+ EXPECT_EQ("visited changed to true", change_log[1]);
+ EXPECT_EQ("checkedState changed from 2 to 1", change_log[2]);
tree.SetDelegate(NULL);
}
« no previous file with comments | « ui/accessibility/ax_node_position_unittest.cc ('k') | ui/accessibility/platform/ax_platform_node_auralinux.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698