AX checked state changes

ash/common/system/chromeos/ime_menu/ime_list_view.cc

Index: ash/common/system/chromeos/ime_menu/ime_list_view.cc
diff --git a/ash/common/system/chromeos/ime_menu/ime_list_view.cc b/ash/common/system/chromeos/ime_menu/ime_list_view.cc
index c26468cad1d3d677cfe222cf182755389a0d99c9..e167836ac1099151903942ee386280750d458f7e 100644
--- a/ash/common/system/chromeos/ime_menu/ime_list_view.cc
+++ b/ash/common/system/chromeos/ime_menu/ime_list_view.cc
@@ -62,8 +62,9 @@ class SelectableHoverHighlightView : public HoverHighlightView {
   void GetAccessibleNodeData(ui::AXNodeData* node_data) override {
     HoverHighlightView::GetAccessibleNodeData(node_data);
     node_data->role = ui::AX_ROLE_CHECK_BOX;
-    if (selected_)
-      node_data->AddStateFlag(ui::AX_STATE_CHECKED);
+    const int checkedState =

[dmazzoni, 2017/03/09 19:33:41]

const ui::AXCheckedState checkedState (instead of int)

[aleventhal, 2017/03/09 21:40:49]

Done.

+        selected_ ? ui::AX_CHECKED_STATE_TRUE : ui::AX_CHECKED_STATE_FALSE;
+    node_data->AddIntAttribute(ui::AX_ATTR_CHECKED_STATE, checkedState);
   }
 
  private:
@@ -157,8 +158,9 @@ class ImeListItemView : public ActionableView {
   void GetAccessibleNodeData(ui::AXNodeData* node_data) override {
     ActionableView::GetAccessibleNodeData(node_data);
     node_data->role = ui::AX_ROLE_CHECK_BOX;
-    node_data->AddStateFlag(selected_ ? ui::AX_STATE_CHECKED
-                                      : ui::AX_STATE_NONE);
+    const int checkedState =

[dmazzoni, 2017/03/09 19:33:41]

same

[aleventhal, 2017/03/09 21:40:49]

Done.

+        selected_ ? ui::AX_CHECKED_STATE_TRUE : ui::AX_CHECKED_STATE_FALSE;
+    node_data->AddIntAttribute(ui::AX_ATTR_CHECKED_STATE, checkedState);
   }
 
  private: