Closed Bug 1173708 Opened 5 years ago Closed 5 years ago

Intermittent test_inlinescript.html | there should be 4 inline scripts that ran - got 3, expected 4

Categories

(Core :: DOM: Security, defect)

x86
macOS
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla42
Tracking Status
firefox40 --- unaffected
firefox41 --- fixed
firefox42 --- fixed
firefox-esr31 --- unaffected
firefox-esr38 --- unaffected

People

(Reporter: cbook, Assigned: ckerschb)

References

()

Details

(Keywords: intermittent-failure)

Attachments

(1 file, 1 obsolete file)

mozilla-inbound_snowleopard_test-mochitest-3

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

01:53:46 INFO - 340 INFO TEST-UNEXPECTED-FAIL | dom/security/test/csp/test_inlinescript.html | there should be 4 inline scripts that ran - got 3, expected 4
Need an owner for this very frequent orange.
Flags: needinfo?(sstamm)
(In reply to Ryan VanderMeulen [:RyanVM UTC-4] from comment #52)
> Need an owner for this very frequent orange.

Sid, I am pretty sure you don't mind that I'll take this one. Nothing obvious sticks out that might go wrong here, but I'll investigate. If you have any thoughts, please let me know.
Assignee: nobody → mozilla
Flags: needinfo?(sstamm)
Blocks: 515442
Franziskus is going to look into that problem. Maybe as a first step, just add some JS debugging statements and push to try. Thanks for taking on the work here!
Assignee: mozilla → franziskuskiefer
Flags: needinfo?(franziskuskiefer)
As discussed in person yesterday it would probably be easiest to have the two individual tests run sequentially (first check that inline scripts are allowed, then check that inline scripts are blocked).

When rewriting the test it might also make sense to remove the ^header^ files and one of the duplicate files needed to serve different headers. It would be nicer to serve the file using different policies using the file_testserver.sjs.

Thanks for taking on the work Franciskus!
(In reply to Christoph Kerschbaumer [:ckerschb] from comment #200)
> Thanks for taking on the work Franciskus!

s/Franciskus/Franziskus :-)