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

Unified Diff: ui/views/controls/table/table_view.cc

Issue 2719223002: Use correct position when setting pos_in_set for tabs.
Patch Set: Potential nit. Created 3 years, 10 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/views/controls/table/table_view.cc
diff --git a/ui/views/controls/table/table_view.cc b/ui/views/controls/table/table_view.cc
index ab17e37d723aad00bee65cb576e12a2cc833c984..d747e021259f2288a9b15bff894581c2b3f3e8ab 100644
--- a/ui/views/controls/table/table_view.cc
+++ b/ui/views/controls/table/table_view.cc
@@ -443,8 +443,8 @@ void TableView::GetAccessibleNodeData(ui::AXNodeData* node_data) {
// Get information about the active item, this is not the same as the set
// of selected items (of which there could be more than one).
node_data->role = ui::AX_ROLE_ROW;
- node_data->AddIntAttribute(ui::AX_ATTR_POS_IN_SET,
- selection_model_.active());
+ node_data->AddValidatedIntAttribute(ui::AX_ATTR_INDEX_IN_SET,
+ selection_model_.active());
if (selection_model_.IsSelected(selection_model_.active())) {
node_data->AddStateFlag(ui::AX_STATE_SELECTED);
}
« chrome/browser/ui/views/tabs/tab_strip.cc ('K') | « ui/views/controls/combobox/combobox.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698