Open Bug 558589 Opened 10 years ago Updated 8 years ago

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

Categories

(Core :: Disability Access APIs, defect, minor)

defect
Not set
minor

Tracking

()

ASSIGNED

People

(Reporter: sgautherie, Assigned: MarcoZ)

References

(Blocks 1 open bug)

Details

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.
Blocks: 558614
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
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?
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).
Blocks: 591741
Blocks: 718237
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.
Depends on: 719754
Marco, could you take this bug?
Taking it!
Assignee: nobody → marco.zehe
Status: NEW → ASSIGNED
You need to log in before you can comment on or make changes to this bug.