1143314 - The single_transition test in dom/animation/test/chrome/test_animation_observers.html fails intermittently

Reporter

Description

•

10 years ago

Since https://hg.mozilla.org/integration/mozilla-inbound/rev/94fa4a005c33 landed on mozilla-inbound last night, there's been a frequent intermittent failure of the form: 652 INFO TEST-UNEXPECTED-FAIL | dom/animation/test/chrome/test_animation_observers.html | [single_transition] records after transition start - number of records - got 2, expected 1 654 INFO TEST-UNEXPECTED-FAIL | dom/animation/test/chrome/test_animation_observers.html | [single_transition] records after transition end - number of records - got 0, expected 1 I think it looks like the 0.001s transition hack that it's using isn't reliable in terms of not finishing before the middle of the test. I'm probably going to disable this test when the tree closing window ends.

David Baron :dbaron:

Reporter

Comment 1

•

10 years ago

I disabled the tests that were failing here: https://hg.mozilla.org/integration/mozilla-inbound/rev/3f0355067f0f

Flags: needinfo?(cam)

Comment hidden (Legacy TBPL/Treeherder Robot)

log: https://treeherder.mozilla.org/logviewer.html#?repo=mozilla-inbound&job_id=7622788 repository: mozilla-inbound start_time: 2015-03-14T00:31:10 who: dbaron[at]dbaron[dot]org machine: t-yosemite-r5-0026 buildname: Rev5 MacOSX Yosemite 10.10 mozilla-inbound debug test mochitest-other revision: ec3e39670ac9 652 INFO TEST-UNEXPECTED-FAIL | dom/animation/test/chrome/test_animation_observers.html | [single_transition] records after transition start - number of records - got 2, expected 1 654 INFO TEST-UNEXPECTED-FAIL | dom/animation/test/chrome/test_animation_observers.html | [single_transition] records after transition end - number of records - got 0, expected 1 [Child 2225] ###!!! ABORT: Aborting on channel error.: file /builds/slave/m-in-osx64-d-00000000000000000/build/src/ipc/glue/MessageChannel.cpp, line 1589 [Child 2225] ###!!! ABORT: Aborting on channel error.: file /builds/slave/m-in-osx64-d-00000000000000000/build/src/ipc/glue/MessageChannel.cpp, line 1589 2560 INFO TEST-UNEXPECTED-FAIL | toolkit/components/aboutmemory/tests/test_aboutmemory5.xul | uncaught exception - NS_ERROR_FAILURE: Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsIFrameScriptLoader.loadFrameScript] at chrome://mochitests/content/chrome/toolkit/components/aboutmemory/tests/test_aboutmemory5.xul:56 2564 INFO TEST-UNEXPECTED-ERROR | toolkit/components/aboutmemory/tests/test_aboutmemory5.xul | This test left crash dumps behind, but we weren't expecting it to! 2566 INFO TEST-UNEXPECTED-FAIL | toolkit/components/aboutmemory/tests/test_aboutmemory5.xul | undefined assertion name - Result logged after SimpleTest.finish() 2569 INFO TEST-UNEXPECTED-FAIL | toolkit/components/aboutmemory/tests/test_aboutmemory6.xul | two log entries generated - expected PASS 2570 INFO TEST-UNEXPECTED-FAIL | toolkit/components/aboutmemory/tests/test_aboutmemory6.xul | uncaught exception - TypeError: logNodes.childNodes[1] is undefined at chrome://mochitests/content/chrome/toolkit/components/aboutmemory/tests/test_aboutmemory6.xul:63 2617 INFO TEST-UNEXPECTED-FAIL | toolkit/components/aboutmemory/tests/test_memoryReporters.xul | http URLs are anonymized when necessary. Failed case: explicit/window-objects/top(none)/detached/window(data:application\vnd.mozilla.xul+xml,<?xml-stylesheet%20href='chrome:\\global\skin'%20type='text\css'?><window%20xmlns='http:\\www.mozilla.org\keymaster\gatekeeper\there.is.only.xul'%20align='start'%20pack='start'%20style='-moz-appearance:%20none;%20margin:%200;%20padding:%200; PROCESS-CRASH | Main app process exited normally | application crashed [@ mozalloc_abort(char const*)] TEST-UNEXPECTED-FAIL | leakcheck | tab process: missing output line for total leaks! Return code: 1

Comment hidden (Legacy TBPL/Treeherder Robot)

David Baron :dbaron:

Reporter

Updated

•

10 years ago

Keywords: leave-open

Cameron McCormack (:heycam)

Assignee

Comment 15

•

10 years ago

Attached patch patch (obsolete) (deleted) — Details — Splinter Review

This makes the transition long and sets currentTime to just before it so that it can still end naturally.

Assignee: nobody → cam

Status: NEW → ASSIGNED

Flags: needinfo?(cam)

Attachment #8577679 - Flags: review?(bbirtles)

Cameron McCormack (:heycam)

Assignee

Updated

•

10 years ago

Attachment #8577679 - Attachment is patch: true

Phil Ringnalda (:philor)

Comment 16

•

10 years ago

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

David Baron :dbaron:

Reporter

Updated

•

10 years ago

Keywords: leave-open

Comment hidden (Legacy TBPL/Treeherder Robot)

Cameron McCormack (:heycam)

Assignee

Comment 19

•

10 years ago

I guess I should fix that for the single_animation subtest, too.

Cameron McCormack (:heycam)

Assignee

Updated

•

10 years ago

Attachment #8577679 - Attachment is obsolete: true

Attachment #8577679 - Flags: review?(bbirtles)

Comment hidden (Legacy TBPL/Treeherder Robot)

Cameron McCormack (:heycam)

Assignee

Comment 21

•

10 years ago

Attached patch fix v2 (deleted) — Details — Splinter Review

https://treeherder.mozilla.org/#/jobs?repo=try&revision=9c0d5bb396fa

Attachment #8578364 - Flags: review?(bbirtles)

Cameron McCormack (:heycam)

Assignee

Updated

•

10 years ago

Attachment #8578364 - Attachment is patch: true

Cameron McCormack (:heycam)

Assignee

Comment 22

•

10 years ago

Also removed a stray dump() call I left in there.

Cameron McCormack (:heycam)

Assignee

Comment 23

•

10 years ago

Sorry, this one, with a correct trychooser test name: https://treeherder.mozilla.org/#/jobs?repo=try&revision=1b81023d946c

Comment hidden (Legacy TBPL/Treeherder Robot)

Brian Birtles (:birtles)

Updated

•

10 years ago

Attachment #8578364 - Flags: review?(bbirtles) → review+

Cameron McCormack (:heycam)

Assignee

Comment 25

•

10 years ago

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

Comment hidden (Legacy TBPL/Treeherder Robot)

Ryan VanderMeulen [:RyanVM]

Comment 32

•

10 years ago

https://hg.mozilla.org/mozilla-central/rev/a80676087e0a

Status: ASSIGNED → RESOLVED

Closed: 10 years ago

status-firefox39: --- → fixed

Flags: in-testsuite+

Resolution: --- → FIXED

Target Milestone: --- → mozilla39

Ryan VanderMeulen [:RyanVM]

Updated

•

10 years ago

status-firefox37: --- → unaffected

status-firefox38: --- → unaffected

status-firefox-esr31: --- → unaffected

Comment hidden (Intermittent Failures Robot)

patch 10 years ago Cameron McCormack (:heycam) (deleted), patch		Details \| Diff \| Splinter Review
fix v2 10 years ago Cameron McCormack (:heycam) (deleted), patch	birtles : review+	Details \| Diff \| Splinter Review