1340504 - Enable EventListener-invoke-legacy.html

Status: RESOLVED FIXED

(Testing :: web-platform-tests, defect)

Comment 1

[Xidorn Quan [:xidorn] UTC+11]

Why was this test disabled? It seems it was disabled in bug 1308969 but I see no justification there at all.

Comment 2

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

It hit

TEST-UNEXPECTED-TIMEOUT | /dom/events/EventListener-invoke-legacy.html | Listener of animationiteration - Test timed out
TEST-UNEXPECTED-TIMEOUT | /dom/events/EventListener-invoke-legacy.html | Legacy listener of animationiteration - Test timed out
TEST-UNEXPECTED-TIMEOUT | /dom/events/EventListener-invoke-legacy.html | expected OK

when bug 1308969 first landed in <https://hg.mozilla.org/integration/mozilla-inbound/rev/65467acedb74>.

I wanted to avoid having the entire push backed out over that, so I temporarily disabled the test. My mistake as far as the bug number is concerned; I meant to refer to bug 1340474.

Comment 3

[Xidorn Quan [:xidorn] UTC+11]

Looks like a low-frequent windows-only intermittent. It doesn't seem to me we should disable it.

https://treeherder.mozilla.org/#/jobs?repo=try&revision=4a317ae79488d6bec7b11dddd5946107a789ca4c

Comment 4

[Xidorn Quan [:xidorn] UTC+11]

Attachment: Bug 1340504 - Enable EventListener-invoke-legacy.

Review commit: https://reviewboard.mozilla.org/r/122294/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/122294/

Comment 5

[James Graham [:jgraham]]

Comment on attachment 8849501 [details]
Bug 1340504 - Enable EventListener-invoke-legacy.

https://reviewboard.mozilla.org/r/122294/#review126760

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/33ee338152e6
Enable EventListener-invoke-legacy. r=jgraham

Comment 7

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/33ee338152e6

Comment 8

[Joel Maher ( :jmaher ) (UTC -8)]

I just submitted a patch to disable this in bug 1351409 :(