Closed Bug 1400487 Opened 2 years ago Closed 2 years ago

Intermittent dom/security/test/csp/test_meta_element.html | Test timed out.

Categories

(Core :: DOM: Security, defect, P5)

defect

Tracking

()

RESOLVED FIXED
mozilla61
Tracking Status
firefox60 --- fixed
firefox61 --- fixed

People

(Reporter: intermittent-bug-filer, Assigned: vinoth)

Details

(Keywords: intermittent-failure, Whiteboard: [stockwell fixed:other][domsecurity-intermittent])

Attachments

(1 file)

Whiteboard: [domsecurity-intermittent]
In the last 7 days, there have been 36 failures.
The failures are on Linux, Linux x64, windows10-64 but a few also on Windows 7 and linux64-qr.
Affected build types: opt and pgo.

An example of a recent log file:
https://treeherder.mozilla.org/logviewer.html#?repo=mozilla-inbound&job_id=166915248&lineNumber=2112

And the relevant part of the log:
03:15:09     INFO -  125 INFO TEST-START | dom/security/test/csp/test_meta_element.html
03:20:26     INFO -  TEST-INFO | started process screenshot
03:20:26     INFO -  TEST-INFO | screenshot: exit 0
03:20:26    ERROR -  126 INFO TEST-UNEXPECTED-FAIL | dom/security/test/csp/test_meta_element.html | Test timed out.
03:20:26     INFO -      reportError@SimpleTest/TestRunner.js:121:7
03:20:26     INFO -      TestRunner._checkForHangs@SimpleTest/TestRunner.js:142:7
03:20:26     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
03:20:26     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
03:20:26     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
03:20:26     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
03:20:26     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
03:20:26     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
03:20:26     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
03:20:26     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
03:20:26     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
03:20:26     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
03:20:26     INFO -      setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
03:20:26     INFO -      TestRunner.runTests@SimpleTest/TestRunner.js:380:5
03:20:26     INFO -      RunSet.runtests@SimpleTest/setup.js:194:3
03:20:26     INFO -      RunSet.runall@SimpleTest/setup.js:173:5
03:20:26     INFO -      hookupTests@SimpleTest/setup.js:266:5
03:20:26     INFO -  parseTestManifest@http://mochi.test:8888/manifestLibrary.js:36:5

:ckerschb As you are the triage owner of this component, could you please take a look at this?
Thank you!
Flags: needinfo?(ckerschb)
Whiteboard: [domsecurity-intermittent] → [stockwell needswork][domsecurity-intermittent]
:vino, can you take a look at this one as well please? Maybe there is an easy fix -thanks!
Assignee: nobody → cegvinoth
Flags: needinfo?(ckerschb)
(In reply to Christoph Kerschbaumer [:ckerschb] from comment #6)
> :vino, can you take a look at this one as well please? Maybe there is an
> easy fix -thanks!

Yes will take a look at it.
This Bug has failed 33 times in the last 7 days on Linux affecting opt, pgo, asan, Windows 7 pgo and opt, Windows 10 opt, pgo build type.
Failing tests: opt-mochitest-e10s, opt-mochitest-plain-headless-e10.

Link to recent log: https://treeherder.mozilla.org/logviewer.html#?repo=mozilla-central&job_id=168486569&lineNumber=2135

:vinoth do you have any update on this?
Flags: needinfo?(cegvinoth)
There are 33 failures in the past week.
Platforms: 
- Linux x64 opt and pgo
- linux64-qr opt
- Windows 7 opt, pgo and debug
- windows10-64 opt, pgo and debug
- windows10-64-nightly opt
- Linux opt
Recent log failure: https://treeherder.mozilla.org/logviewer.html#?repo=autoland&job_id=170683553&lineNumber=2339
Relevant part of the log: 
[task 2018-03-27T23:15:42.276Z] 23:15:42     INFO - TEST-INFO | screentopng: exit 0
[task 2018-03-27T23:15:42.277Z] 23:15:42     INFO - TEST-UNEXPECTED-FAIL | dom/security/test/csp/test_meta_element.html | Test timed out. 
[task 2018-03-27T23:15:42.278Z] 23:15:42     INFO -     reportError@SimpleTest/TestRunner.js:121:7
[task 2018-03-27T23:15:42.279Z] 23:15:42     INFO -     TestRunner._checkForHangs@SimpleTest/TestRunner.js:142:7
[task 2018-03-27T23:15:42.280Z] 23:15:42     INFO -     setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
[task 2018-03-27T23:15:42.280Z] 23:15:42     INFO -     setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
[task 2018-03-27T23:15:42.281Z] 23:15:42     INFO -     setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
[task 2018-03-27T23:15:42.282Z] 23:15:42     INFO -     setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
[task 2018-03-27T23:15:42.283Z] 23:15:42     INFO -     setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
[task 2018-03-27T23:15:42.283Z] 23:15:42     INFO -     setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
[task 2018-03-27T23:15:42.284Z] 23:15:42     INFO -     setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
[task 2018-03-27T23:15:42.284Z] 23:15:42     INFO -     setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
[task 2018-03-27T23:15:42.286Z] 23:15:42     INFO -     setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
[task 2018-03-27T23:15:42.286Z] 23:15:42     INFO -     setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
[task 2018-03-27T23:15:42.287Z] 23:15:42     INFO -     setTimeout handler*TestRunner._checkForHangs@SimpleTest/TestRunner.js:163:5
[task 2018-03-27T23:15:42.288Z] 23:15:42     INFO -     TestRunner.runTests@SimpleTest/TestRunner.js:380:5
[task 2018-03-27T23:15:42.288Z] 23:15:42     INFO -     RunSet.runtests@SimpleTest/setup.js:194:3
[task 2018-03-27T23:15:42.288Z] 23:15:42     INFO -     RunSet.runall@SimpleTest/setup.js:173:5
[task 2018-03-27T23:15:42.289Z] 23:15:42     INFO -     hookupTests@SimpleTest/setup.js:266:5
[task 2018-03-27T23:15:42.289Z] 23:15:42     INFO - parseTestManifest@http://mochi.test:8888/manifestLibrary.js:36:5
[task 2018-03-27T23:15:42.290Z] 23:15:42     INFO - getTestManifest/req.onload@http://mochi.test:8888/manifestLibrary.js:49:11
[task 2018-03-27T23:15:42.291Z] 23:15:42     INFO - EventHandlerNonNull*getTestManifest@http://mochi.test:8888/manifestLibrary.js:45:3
[task 2018-03-27T23:15:42.292Z] 23:15:42     INFO -     hookup@SimpleTest/setup.js:246:5
:vinoth Can you please take a look here?
(In reply to Noemi Erli[:noemi_erli] from comment #12)
> hookup@SimpleTest/setup.js:246:5
> :vinoth Can you please take a look here?

Ya I am working on it.
Flags: needinfo?(cegvinoth)
Comment on attachment 8963541 [details]
Bug 1400487 - Moved img src declaration after onload and onerror declaration.

Main issue is, Onload and Onerror event functions should be declared before loading the image.
Previously image was loaded earlier, hence in few platforms these event handlers never sent message to the parent window. Hence we got the timeout issue.

Tested the new changes in try server,
https://treeherder.mozilla.org/#/jobs?repo=try&revision=6c52bed5df9a6496dddf86bb3769c4be7766de04&selectedJob=170979623
Attachment #8963541 - Flags: review?(ckerschb)
Comment on attachment 8963541 [details]
Bug 1400487 - Moved img src declaration after onload and onerror declaration.

Christoph Kerschbaumer [:ckerschb] has approved the revision.

https://phabricator.services.mozilla.com/D828
Attachment #8963541 - Flags: review+
Comment on attachment 8963541 [details]
Bug 1400487 - Moved img src declaration after onload and onerror declaration.

some phabricator issue on updating the flag.
Attachment #8963541 - Flags: review?(ckerschb)
Keywords: checkin-needed
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/483a612a7c5a
Move img src declaration after onload and onerror declaration in file_meta_element.html. r=ckerschb
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/483a612a7c5a
Status: REOPENED → RESOLVED
Closed: 2 years ago2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
Whiteboard: [stockwell needswork][domsecurity-intermittent] → [stockwell fixed:other][domsecurity-intermittent]
You need to log in before you can comment on or make changes to this bug.