943269 - Intermittent TEST-UNEXPECTED-FAIL | /tests/layout/forms/test/test_bug402198.html | application timed out after 330 seconds with no output

Status: RESOLVED FIXED

(Core :: Layout: Form Controls, defect, P5)

Description

[Carsten Book [:Tomcat]]

b2g_ubuntu64_vm b2g-inbound opt test mochitest-1 on 2013-11-25 07:16:41 PST for push 261e064c07d7

slave: tst-linux64-ec2-066

https://tbpl.mozilla.org/php/getParsedLog.php?id=31045645&tree=B2g-Inbound

TEST-UNEXPECTED-FAIL | /tests/layout/forms/test/test_bug402198.html | application timed out after 330 seconds with no output
TEST-UNEXPECTED-FAIL | /tests/layout/forms/test/test_bug402198.html | application terminated with exit code 1536

Comment 1

[Mats Palmgren (inactive)]

http://mxr.mozilla.org/mozilla-central/source/layout/forms/test/test_bug402198.html?force=1

59   SimpleTest.finish();
60   setTimeout(function() {document.body.style.display = '';}, 400);

That looks strange; I don't think we should be doing things after calling finish().
I don't see how it could cause a hang though.
Anyway, I don't see "this is a crash/assertion test..." in the log so I guess
we didn't reach line 58, unless we failed to flush that out before we were
killed by the framework.

Fwiw, we should probably change the above to:
setTimeout(function() {document.body.style.display = ''; SimpleTest.finish();}, 400);

Comment 2

[Mats Palmgren (inactive)]

Line 60 wasn't in the original check-in, it was added in:
http://hg.mozilla.org/mozilla-central/diff/37dd897d1c32/layout/forms/test/test_bug402198.html

Martijn, do you remember why you added that line? and do you think the suggested change above
would work?

Comment 3

[Legacy TBPL/Treeherder Robot]

philor
https://tbpl.mozilla.org/php/getParsedLog.php?id=32438975&tree=Cedar
b2g_ubuntu32_vm cedar opt test mochitest-1 on 2013-12-30 20:48:53
revision: 1802ef305927
slave: tst-linux32-spot-091

TEST-UNEXPECTED-FAIL | /tests/layout/forms/test/test_bug402198.html | application timed out after 330 seconds with no output
TEST-UNEXPECTED-FAIL | /tests/layout/forms/test/test_bug402198.html | application terminated with exit code 6
PROCESS-CRASH | /tests/layout/forms/test/test_bug402198.html | application crashed [@ linux-gate.so + 0x424]
Return code: 1

Comment 4

[Legacy TBPL/Treeherder Robot]

philor
https://tbpl.mozilla.org/php/getParsedLog.php?id=33996396&tree=Mozilla-Inbound
b2g_ubuntu32_vm mozilla-inbound opt test mochitest-1 on 2014-02-02 21:54:30
revision: f3c918a0e91c
slave: tst-linux32-spot-159

TEST-UNEXPECTED-FAIL | /tests/layout/forms/test/test_bug402198.html | application timed out after 330 seconds with no output
TEST-UNEXPECTED-FAIL | /tests/layout/forms/test/test_bug402198.html | application terminated with exit code 6
PROCESS-CRASH | /tests/layout/forms/test/test_bug402198.html | application crashed [@ linux-gate.so + 0x424]
Return code: 1

Comment 5

[Legacy TBPL/Treeherder Robot]

RyanVM
https://tbpl.mozilla.org/php/getParsedLog.php?id=35725937&tree=B2g-Inbound
b2g_ubuntu32_vm b2g-inbound opt test mochitest-1 on 2014-03-06 07:29:58
revision: 45a7ba51f378
slave: tst-linux32-spot-065

TEST-UNEXPECTED-FAIL | /tests/layout/forms/test/test_bug402198.html | application timed out after 330 seconds with no output
TEST-UNEXPECTED-FAIL | /tests/layout/forms/test/test_bug402198.html | application terminated with exit code 6
PROCESS-CRASH | /tests/layout/forms/test/test_bug402198.html | application crashed [@ linux-gate.so + 0x424]
Return code: 1

Comment 6

[Martijn Wargers (dead)]

(In reply to Mats Palmgren (:mats) from comment #2)
> Line 60 wasn't in the original check-in, it was added in:
> http://hg.mozilla.org/mozilla-central/diff/37dd897d1c32/layout/forms/test/
> test_bug402198.html
> 
> Martijn, do you remember why you added that line? and do you think the
> suggested change above
> would work?

I think I added it while I was running it as a standalone mochitest, to be able to see the result of the mochitest. I was afraid it would otherwise interfere with the test (in the sense that it wouldn't crash when this issue still occured in the browser).
This shouldn't cause these failures. Anyway, the last time this happened (4 monts ago), it wasn't a timeout, but a crash.

Comment 7

[Mats Palmgren (inactive)]

Attachment: fix

OK, I think we should try this then.

(The crash in comment 5 has "exit code 6" so I'm pretty sure
it's the test framework that intentionally aborted the process
when it detected the timeout.)

https://tbpl.mozilla.org/?tree=Try&rev=c39c56451a28

Comment 8

[Martijn Wargers (dead)]

Comment on attachment 8454909 [details] [diff] [review]
fix

Review of attachment 8454909 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks, that should be fine. However, like I said, I don't think it should fix this intermittent failure.

Comment 9

[Mats Palmgren (inactive)]

(In reply to Martijn Wargers [:mwargers] (QA) from comment #8)
> However, like I said, I don't think it should
> fix this intermittent failure.

Yeah, maybe not, we'll see.

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

Comment 10

[Carsten Book [:Tomcat]]

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

Comment 11

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/0e974612773f
https://hg.mozilla.org/releases/mozilla-beta/rev/1772e55568e4