746531 - Unexpected focus event on testing down key on ml_marmalade in tests_focus_listcontrols.xul

Status: RESOLVED FIXED

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

Description

[Marco Zehe (:MarcoZ)]

Log file:
3958 INFO TEST-INFO | chrome://mochitests/content/a11y/accessible/events/test_focus_listcontrols.xul | Invoke the ' 'menulist'  'down ' key' test { unexpected 'focus' event; unexpected 'focus' event;  }
3959 INFO TEST-PASS | chrome://mochitests/content/a11y/accessible/events/test_focus_listcontrols.xul | test with ID = ' 'menulist'  'down ' key' failed. There is unexpected focus event.
3960 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/a11y/accessible/events/test_focus_listcontrols.xul | test with ID = ' 'menulist'  'down ' key' failed. There is unexpected focus event.
3961 INFO TEST-END | chrome://mochitests/content/a11y/accessible/events/test_focus_listcontrols.xul | finished in 6496ms

Comment 1

[Marco Zehe (:MarcoZ)]

Oops, I wanted to copy these two lines prior to the ones I pasted above:
3956 INFO TEST-PASS | chrome://mochitests/content/a11y/accessible/events/test_focus_listcontrols.xul | test with ID = ' 'ml_marmalade'  'escape ' key' failed. No focus event.
3957 INFO TEST-PASS | chrome://mochitests/content/a11y/accessible/events/test_focus_listcontrols.xul | test with ID = ' 'ml_marmalade'  'escape ' key' failed. There is unexpected focus event.

Comment 2

[Marco Zehe (:MarcoZ)]

Attachment: Disable offending tests

Comment 3

[Marco Zehe (:MarcoZ)]

Disabled test on Mac: http://hg.mozilla.org/integration/mozilla-inbound/rev/e23e7fe11364

Comment 4

[Ed Morley [:emorley]]

(In reply to Marco Zehe (:MarcoZ) from comment #3)
> Disabled test on Mac:
> http://hg.mozilla.org/integration/mozilla-inbound/rev/e23e7fe11364

Sorry, push backed out for linux mochitest-other orange:
https://tbpl.mozilla.org/?tree=Mozilla-Inbound&onlyunstarred=1&rev=8cc13f37bc66
https://tbpl.mozilla.org/php/getParsedLog.php?id=11222399&tree=Mozilla-Inbound

{
4512 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/a11y/accessible/events/test_focus_listcontrols.xul | Error in test: proposed current item 'ml_tangerine' is already current
}

https://hg.mozilla.org/integration/mozilla-inbound/rev/922510fc28aa

Comment 5

[Marco Zehe (:MarcoZ)]

Relanded with bustage fix: http://hg.mozilla.org/integration/mozilla-inbound/rev/175a8c81e3ff

Comment 6

[Morgan Reschenberg [:morgan]]

:eeejay this file doesn't seem to exist anymore. Do you have any intuition re: if it was renamed or moved, where it might live?

Comment 7

[Eitan Isaacson [:eeejay]]

Looks like it is now named accessible/tests/mochitest/events/test_focus_listcontrols.xhtml

There was a mass rename of xul to xhtml here:
https://hg.mozilla.org/mozilla-central/rev/d933287860cb

Comment 8

[Morgan Reschenberg [:morgan]]

Attachment: Bug 746531: Remove Mac-specific casing in listcontrols focus test. r=eeejay

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/4c30a8e81151
Remove Mac-specific casing in listcontrols focus test. r=eeejay

Comment 10

[Stefan Hindli [:stefan_hindli]]

https://hg.mozilla.org/mozilla-central/rev/4c30a8e81151

Comment 11

[Marco Zehe (:MarcoZ)]

Morgan, can this be closed now, or does this need additional work?

Comment 12

[Morgan Reschenberg [:morgan]]

Closing! :) thanks for the ping