599258 - ERROR TEST-UNEXPECTED-FAIL | /tests/content/html/content/test/test_bug561636.html

Status: RESOLVED FIXED

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

Description

[Doug Turner (:dougt)]

This test list up:

50018 ERROR TEST-UNEXPECTED-FAIL | /tests/content/html/content/test/test_bug561636.html | Test timed out.

after adding IME event support to content fake widget:


http://hg.mozilla.org/mozilla-central/rev/1ba6bee99c20
http://hg.mozilla.org/mozilla-central/rev/832f91c1d5a6
http://hg.mozilla.org/mozilla-central/rev/0af6ccc89e51
http://hg.mozilla.org/mozilla-central/rev/cea9bd7e6077
http://hg.mozilla.org/mozilla-central/rev/ba77f120b5e9
http://hg.mozilla.org/mozilla-central/rev/beb5d04777bc
http://hg.mozilla.org/mozilla-central/rev/834f8bf2bf08
http://hg.mozilla.org/mozilla-central/rev/e0f37aadafd1

See bug 591047


http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1285302413.1285304862.10469.gz

Comment 1

[Doug Turner (:dougt)]

00:01 < volkmar> it looks like the last form is never used
00:02 < volkmar> so the same form is used twice which may provec double form submissions some times breaking note think preventing one event preventing the 
                 finish() call

Comment 2

[Mounir Lamouri (:mounir)]

Attachment: Patch v1

This test was a mess!
I forgot to call the test at onload which make the onload event for the second iframe showing up mostly all the time _after_ the addEventListener and making the test finishing green.
But, if the load event was fired before the addEventListener, there were no way to have the test finishing.

In addition, the last .submit() was not done on the correct form.

Damn, I'm not proud of that :(

Comment 3

[Mounir Lamouri (:mounir)]

Attachment: Patch v1

Better with the correct patch....

Comment 4

[Doug Turner (:dougt)]

mounir, thanks for the fast turn around!

Comment 5

[Mounir Lamouri (:mounir)]

http://hg.mozilla.org/mozilla-central/rev/842d82ff333c