Closed Bug 1027127 Opened 10 years ago Closed 10 years ago

Intermittent test_input-manage.html | Test timed out

Categories

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

Product:
Core
Component:
DOM: Core & HTML
Platform:
ARM
Gonk (Firefox OS)
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla33
Tracking Flags:
Tracking Status
firefox31 --- unaffected
firefox32 --- fixed
firefox33 --- fixed
firefox-esr24 --- unaffected
b2g-v1.4 --- fixed
b2g-v2.0 --- fixed
b2g-v2.1 --- fixed

People

(Reporter: emorley, Assigned: xyuan)

References

Details

(Keywords: intermittent-failure)

Attachments

(1 file, 1 obsolete file)

Remove setTimeout
1.31 KB, patch
xyuan
: review+
Details | Diff | Splinter Review 
b2g_emulator_vm mozilla-inbound debug test mochitest-debug-10 on 2014-06-18 04:13:27 PDT for push 45dd001f865a

slave: tst-linux64-spot-1014

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

{
04:55:04     INFO -  208 INFO TEST-START | /tests/dom/permission/tests/test_input-manage.html
04:55:05     INFO -  [Child 750] WARNING: Transparent content with displayports can be expensive.: file ../../../gecko/layout/base/nsDisplayList.cpp, line 1383
04:55:09     INFO -  [Child 750] WARNING: Transparent content with displayports can be expensive.: file ../../../gecko/layout/base/nsDisplayList.cpp, line 1383
04:55:09     INFO -  [Child 750] WARNING: Transparent content with displayports can be expensive.: file ../../../gecko/layout/base/nsDisplayList.cpp, line 1383
04:55:14     INFO -  [Child 750] WARNING: Failed to retarget HTML data delivery to the parser thread.: file ../../../gecko/parser/html/nsHtml5StreamParser.cpp, line 947
04:55:15     INFO -  [Child 750] WARNING: Transparent content with displayports can be expensive.: file ../../../gecko/layout/base/nsDisplayList.cpp, line 1383
04:55:16     INFO -  [Child 750] WARNING: Transparent content with displayports can be expensive.: file ../../../gecko/layout/base/nsDisplayList.cpp, line 1383
04:55:16     INFO -  [Child 750] WARNING: Transparent content with displayports can be expensive.: file ../../../gecko/layout/base/nsDisplayList.cpp, line 1383
04:55:17     INFO -  [Child 750] WARNING: Transparent content with displayports can be expensive.: file ../../../gecko/layout/base/nsDisplayList.cpp, line 1383
04:55:19     INFO -  [Child 750] WARNING: Transparent content with displayports can be expensive.: file ../../../gecko/layout/base/nsDisplayList.cpp, line 1383
04:55:20     INFO -  [Child 750] WARNING: Subdocument container has no frame: file ../../../gecko/layout/base/nsDocumentViewer.cpp, line 2509
04:55:20     INFO -  [Child 750] WARNING: Failed to retarget HTML data delivery to the parser thread.: file ../../../gecko/parser/html/nsHtml5StreamParser.cpp, line 947
04:55:25     INFO -  [Child 750] WARNING: NS_ENSURE_TRUE(txToRemove) failed: file ../../../../gecko/docshell/shistory/src/nsSHistory.cpp, line 1338
04:55:26     INFO -  [Child 750] WARNING: Subdocument container has no frame: file ../../../gecko/layout/base/nsDocumentViewer.cpp, line 2509
04:55:28     INFO -  [Child 750] WARNING: Failed to retarget HTML data delivery to the parser thread.: file ../../../gecko/parser/html/nsHtml5StreamParser.cpp, line 947
04:55:32     INFO -  ############################### browserElementPanning.js loaded
04:55:32     INFO -  ######################## BrowserElementChildPreload.js loaded
04:55:32     INFO -  [Child 750] WARNING: Subdocument container has no presshell: file ../../../gecko/layout/base/nsDocumentViewer.cpp, line 2492
04:55:39     INFO -  209 INFO TEST-INFO | dumping last 5 message(s)
04:55:39     INFO -  210 INFO TEST-INFO | if you need more context, please use SimpleTest.requestCompleteLog() in your test
04:55:39     INFO -  211 INFO TEST-PASS | /tests/dom/permission/tests/test_input-manage.html | Doesn't have input-manage
04:55:39     INFO -  212 INFO TEST-PASS | /tests/dom/permission/tests/test_input-manage.html | Doesn't have browser
04:55:40     INFO -  213 INFO TEST-PASS | /tests/dom/permission/tests/test_input-manage.html | Received exception!: TypeError: invalid 'instanceof' operand window[this.webidl]
04:55:40     INFO -  214 INFO TEST-PASS | /tests/dom/permission/tests/test_input-manage.html | Has input-manage
04:55:40     INFO -  215 INFO TEST-PASS | /tests/dom/permission/tests/test_input-manage.html | Has browser
04:55:40     INFO -  216 INFO TEST-UNEXPECTED-FAIL | /tests/dom/permission/tests/test_input-manage.html | Test timed out
04:55:40     INFO -  [Child 750] WARNING: NS_ENSURE_TRUE(txToRemove) failed: file ../../../../gecko/docshell/shistory/src/nsSHistory.cpp, line 1338
04:55:41     INFO -  217 INFO TEST-INFO | MEMORY STAT vsize after test: 118218752
04:55:41     INFO -  218 INFO TEST-INFO | MEMORY STAT residentFast after test: 65765376
04:55:41     INFO -  219 INFO TEST-INFO | MEMORY STAT heapAllocated after test: 18884824
04:55:43     INFO -  220 INFO TEST-END | /tests/dom/permission/tests/test_input-manage.html | finished in 38007ms
}
Mind taking a look, Yuan? :)
Flags: needinfo?(xyuan)
Otherwise, might this be similar to bug 1025994?
It should be the same as bug 1030080. The test waits for the load event of a mozbrowser, which is not fired sometimes.
Flags: needinfo?(xyuan)
Attached patch removeTimeout.patch (obsolete) — — Splinter Review 
After checking the log, I found that it is a false failure of time out. This test sets a time out to report error when the test runs longer than 10s. It is not necessary and the mochitest framework has its own way to handle the time out case. So I removed the unnecessary timer.
Assignee: nobody → xyuan
Status: NEW → ASSIGNED
Blocks: 1030080
Comment on attachment 8447967 [details] [diff] [review]
removeTimeout.patch

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

b2g_emulator_vm is too slow to run the test in 10s. With the 10s timeout removed, the test ran without failure on try server more than 50 times. The patch seems to be working.
Attachment #8447967 - Flags: review?(dchan)
Comment on attachment 8447967 [details] [diff] [review]
removeTimeout.patch

Looks good to me. I forget the original reason I put in the timer, but this change won't affect correctness of any tests.
Attachment #8447967 - Flags: review?(dchan) → review+
Update commit message to add review info.
Attachment #8447967 - Attachment is obsolete: true
Attachment #8449987 - Flags: review+
https://hg.mozilla.org/mozilla-central/rev/559eb932b214
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla33
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: