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

Unified Diff: chrome/common/extensions/api/automation.idl

Issue 2799443002: Implemented ARIA colindex, rowindex, colcount and rowcount for Chromevox. (Closed)
Patch Set: Fixed automation test. 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: chrome/common/extensions/api/automation.idl
diff --git a/chrome/common/extensions/api/automation.idl b/chrome/common/extensions/api/automation.idl
index b10bf511e21137e3af23554edaab4d7461ffc347..cd074557af5c48dc0a3a1a1ed3048161dcf95665 100644
--- a/chrome/common/extensions/api/automation.idl
+++ b/chrome/common/extensions/api/automation.idl
@@ -552,25 +552,37 @@
// Table attributes.
//
- // The number of rows in this table.
+ // The number of rows in this table as specified in the DOM.
long? tableRowCount;
- // The number of columns in this table.
+ // The number of rows in this table as specified by the page author.
+ long? ariaRowCount;
+
+ // The number of columns in this table as specified in the DOM.
long? tableColumnCount;
+ // The number of columns in this table as specified by the page author.
+ long? ariaColumnCount;
+
//
// Table cell attributes.
//
- // The zero-based index of the column that this cell is in.
+ // The zero-based index of the column that this cell is in as specified in the DOM.
michaelpg 2017/04/06 14:52:05 nit: wrap line at 80 chars
long? tableCellColumnIndex;
+ // The ARIA column index as specified by the page author.
+ long? ariaCellColumnIndex;
+
// The number of columns that this cell spans (default is 1).
long? tableCellColumnSpan;
- // The zero-based index of the row that this cell is in.
+ // The zero-based index of the row that this cell is in as specified in the DOM.
michaelpg 2017/04/06 14:52:05 nit: wrap line at 80 chars
long? tableCellRowIndex;
+ // The ARIA row index as specified by the page author.
+ long? ariaCellRowIndex;
+
// The number of rows that this cell spans (default is 1).
long? tableCellRowSpan;

Powered by Google App Engine
This is Rietveld 408576698