Closed Bug 1245560 Opened 8 years ago Closed 8 years ago

Intermittent test_presentation_receiver_oop.html | Test timed out. and more presentation test failures.

Categories

(Core :: DOM: Device Interfaces, defect)

defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 1263878

People

(Reporter: aryx, Assigned: schien)

Details

(Keywords: intermittent-failure)

https://treeherder.mozilla.org/logviewer.html#?job_id=20994413&repo=mozilla-inbound

17:08:04     INFO -  383 INFO TEST-PASS | dom/presentation/tests/mochitest/test_presentation_receiver_oop.html | Message from iframe: OK navigator.presentation shouldn't be available in non-receiving OOP pages.
17:08:04     INFO -  384 INFO TEST-PASS | dom/presentation/tests/mochitest/test_presentation_receiver_oop.html | Messaging from iframe complete.
17:08:04     INFO -  385 INFO TEST-PASS | dom/presentation/tests/mochitest/test_presentation_receiver_oop.html | Message from iframe: OK Connection ID should be set: test-session-id
17:08:04     INFO -  386 INFO TEST-PASS | dom/presentation/tests/mochitest/test_presentation_receiver_oop.html | Message from iframe: OK Connection state at receiver side should be closed by default.
17:08:04     INFO -  387 INFO An offer is received.
17:08:04     INFO -  388 INFO TEST-PASS | dom/presentation/tests/mochitest/test_presentation_receiver_oop.html | A valid answer is sent.
17:08:04     INFO -  389 INFO TEST-UNEXPECTED-FAIL | dom/presentation/tests/mochitest/test_presentation_receiver_oop.html | Test timed out.
17:08:04     INFO -      reportError@SimpleTest/TestRunner.js:114:7
17:08:04     INFO -      TestRunner._checkForHangs@SimpleTest/TestRunner.js:134:7
17:08:04     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:155:5
17:08:04     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:155:5
17:08:04     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:155:5
17:08:04     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:155:5
17:08:04     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:155:5
17:08:04     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:155:5
17:08:04     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:155:5
17:08:04     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:155:5
17:08:04     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:155:5
17:08:04     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:155:5
17:08:04     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:155:5
17:08:04     INFO -      TestRunner.runTests@SimpleTest/TestRunner.js:366:5
17:08:04     INFO -      RunSet.runtests@SimpleTest/setup.js:188:3
17:08:04     INFO -      RunSet.runall@SimpleTest/setup.js:167:5
17:08:04     INFO -      hookupTests@SimpleTest/setup.js:260:5
17:08:04     INFO -  parseTestManifest@http://mochi.test:8888/manifestLibrary.js:36:5
17:08:04     INFO -  getTestManifest/req.onload@http://mochi.test:8888/manifestLibrary.js:49:11
17:08:04     INFO -  EventHandlerNonNull*getTestManifest@http://mochi.test:8888/manifestLibrary.js:45:3
17:08:04     INFO -      hookup@SimpleTest/setup.js:240:5
17:08:04     INFO -  EventHandlerNonNull*@http://mochi.test:8888/tests?autorun=1&closeWhenDone=1&consoleLevel=INFO&manifestFile=tests.json&dumpOutputDirectory=%2Ftmp:11:1
17:08:04     INFO -  MEMORY STAT | vsize 1025MB | residentFast 220MB | heapAllocated 69MB
17:08:04     INFO -  390 INFO TEST-OK | dom/presentation/tests/mochitest/test_presentation_receiver_oop.html | took 323924ms
not sure if it related to bug 1245031. need further investigation.
(In reply to Shih-Chiang Chien [:schien] (UTC+8) (use ni? plz) from comment #1)
> not sure if it related to bug 1245031. need further investigation.

Doesn't appear to have helped. I've been trying to bisect this down a bit as well, and I've got it down to a merge at this point: https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=8a6976312049
(In reply to Ryan VanderMeulen [:RyanVM] from comment #6)
> (In reply to Shih-Chiang Chien [:schien] (UTC+8) (use ni? plz) from comment
> #1)
> > not sure if it related to bug 1245031. need further investigation.
> 
> Doesn't appear to have helped. I've been trying to bisect this down a bit as
> well, and I've got it down to a merge at this point:
> https://treeherder.mozilla.org/#/jobs?repo=mozilla-
> inbound&revision=8a6976312049

Doesn't see any obvious bug that directed related to this intermittent failure. Assign myself to investigate it.
Assignee: nobody → schien
Yeah, that proved to be a waste of effort. I was able to reproduce it even further back, but at a much lower frequency. I gave up looking for a smoking gun change because I'm not sure one exists or if it's worth the energy to find at this point.
(In reply to Ryan VanderMeulen [:RyanVM] from comment #9)
> Yeah, that proved to be a waste of effort. I was able to reproduce it even
> further back, but at a much lower frequency. I gave up looking for a smoking
> gun change because I'm not sure one exists or if it's worth the energy to
> find at this point.

That's ok. Maybe there is a timing issue in Presentation API code and I plan to use rr and chaos mode to figure out the root cause.
From the log it looks like the state of PresentationConnection is already 'connected' while executing [1]. Therefore, test is timeout because there is no statechange event fired.

Double checking spec and code to figure out what should be fixed.

[1] https://dxr.mozilla.org/mozilla-central/source/dom/presentation/tests/mochitest/file_presentation_receiver_oop.html#79
There are other intermittent timeout under dom/presentation and might have the same root cause.

test_presentation_sender_disconnect.html
test_presentation_sender.html
test_presentation_sender_default_request.html

All the timeout happened while waiting for device availability change and all happened after the test_presentation_receiver_oop.html timeout.

Pushing a try to figure out if my patch works: https://treeherder.mozilla.org/#/jobs?repo=try&revision=3f1dca5a782a
Teardown procedure is not triggered after test_presentation_receiver_oop.html timeout and that's the root cause of the following test timeout. Fix test_presentation_receiver_oop.html timeout is the key to this intermittent failure.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.