883164 - More test for <input type='file'>.click()

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Mounir Lamouri (:mounir)]

Attachment: Patch

It should be a bit better than having a special test at the end of the tests.

Comment 1

[Mounir Lamouri (:mounir)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/eff8e7fefffd

Comment 2

[Ed Morley [:emorley]]

Backed out since the test now times out on Android:
https://tbpl.mozilla.org/php/getParsedLog.php?id=24227887&tree=Mozilla-Inbound

https://hg.mozilla.org/integration/mozilla-inbound/rev/6e84744ffa9a

Comment 3

[Ed Morley [:emorley]]

Actually not just Android:
https://tbpl.mozilla.org/?tree=Mozilla-Inbound&rev=eff8e7fefffd

Comment 4

[Mounir Lamouri (:mounir)]

Hmmm, last I check that was green locally :(

Comment 5

[Mounir Lamouri (:mounir)]

The reason was that I was testing the test alone and the test is run in an iframe on try and even if I was calling .focus() on each element before clicking, the <div> are not focusable by default. Easily solved with a tabindex='1' on the divs.

https://hg.mozilla.org/integration/mozilla-inbound/rev/3e6afe9129c3

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/3e6afe9129c3