558589 - A11y tests which depend on the "tabbrowser API" should explicitly check its availability.

Status: NEW

(Core :: Disability Access APIs, defect)

Description

[Serge Gautherie (:sgautherie)]

http://mxr.mozilla.org/mozilla-central/source/accessible/tests/mochitest/name_nsRootAcc_wnd.xul
http://mxr.mozilla.org/mozilla-central/source/accessible/tests/mochitest/events/test_scroll.xul
http://mxr.mozilla.org/mozilla-central/source/accessible/tests/mochitest/tree/test_tabbrowser.xul
depend on the "tabbrowser API".

They need to bind to both ("either" would be better, but...)
chrome://browser/content/browser.css
and
chrome://navigator/content/navigator.css

Such tests should detect and "warn" when the "API" they need is not available:
this would make failures caused by API mismatch more obvious.

Comment 1

[Philip Chee]

For test purposes only would it be possible to register an override in the *.manifest e.g.

override chrome://browser/content/browser.css chrome://navigator/content/navigator.css

Comment 2

[[e-mail]]

If they need a tabbrowser, why don't they either a) run as browser-chrome tests b) access the window's tabbrowser in some other way e.g window-watcher?

Comment 3

[Robert Kaiser]

They probably don't run as browser-chrome tests because the a11y folks want to have their tests all together in this suite (also note that there may be platforms/configs where this suite can't be run, but others are).

Comment 4

[Serge Gautherie (:sgautherie)]

I assume bug 719754 is a major (final) step in the right direction :-)
That bug was about 'tabbrowser'.

***

There are 2 other tests, that use 'searchbar',
{
/accessible/tests/mochitest/events/test_focus_autocomplete.xul
    * line 7 -- <?xml-stylesheet href="chrome://browser/content/browser.css"
/accessible/tests/mochitest/states/test_expandable.xul
    * line 7 -- <?xml-stylesheet href="chrome://browser/content/browser.css"
}
which might be good candidates to a similar update.

NB: Hoping that may help wrt bug 718237.

Comment 5

[alexander :surkov (:asurkov)]

Marco, could you take this bug?

Comment 6

[Marco Zehe (:MarcoZ)]

Taking it!

Comment 7

[BugBot [:suhaib / :marco/ :calixte]]

The bug assignee is inactive on Bugzilla, so the assignee is being reset.