New issue
Advanced search Search tips

Issue 901886 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 2
Type: Bug
Team-Accessibility



Sign in to add a comment

In chrome.automation API getter `tableCellRowHeaders` in class AutomationNode is not working

Reported by yura.cha...@gmail.com, Nov 5

Issue description

UserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/70.0.3538.77 Safari/537.36

Steps to reproduce the problem:
1. Create extension with permissions to call Automation API.
2. Call chrome.automation.getTree(...).
3. In a callback access rootNode.tableCellRowHeaders (e.g. console.log(rootNode.tableCellRowHeaders)).

What is the expected behavior?
Return value of rootNode.tableCellRowHeaders.

What went wrong?
Threw an exception:

tableCellRowHeaders: [Exception: ReferenceError: ids is not defined at Object.get tableCellRowHeaders [as tableCellRowHeaders] (extensions::automationNode:540:5) at AutomationRootNode.get [as tableCellRowHeaders] (extensions::utils:166:38) at AutomationRootNode.invokeGetter (<anonymous>:2:14) at chrome.automation.getTree (chrome-extension://nhcahbgndlbgobnohbkhghjagbophbfc/src/bg/background.js:9:30) at extensions::automation:318:5 at EventImpl.dispatchToListener (extensions::event_bindings:403:22) at Event.publicClassPrototype.(anonymous function) [as dispatchToListener] (extensions::utils:138:26) at EventImpl.dispatch_ (extensions::event_bindings:387:35) at dispatchArgs (extensions::event_bindings:260:26) at dispatchEvent (extensions::event_bindings:269:7)]

Did this work before? N/A 

Does this work in other browsers? N/A

Chrome version: 72.0.3601.0  Channel: canary
OS Version: 10.0
Flash Version: 

There is a typo in extensions::automationNode in line 539, should be `ids` instead of `id`.

538:  get tableCellRowHeaders() {
539:    var id = GetTableCellRowHeaders(this.treeID, this.id);
540:    if (ids && this.rootImpl) {
541:      var result = [];
542:      for (var i = 0; i < ids.length; i++)
543:        result.push(this.rootImpl.get(ids[i]));
544:      return result;
545:    }
546:  },
 
Components: UI>Accessibility
Owner: dmazz...@chromium.org
Status: Assigned (was: Unconfirmed)
Thanks for the report. I confirm that this typo is in ToT...

Was added in this CL.

https://chromium.googlesource.com/chromium/src/+/d42e00a2b908b813aa98b35a0ceef685f3b316cf

Sign in to add a comment