Intermittent test_CSP_bug885433.html | Inline script should be allowed - Result logged after SimpleTest.finish()

RESOLVED FIXED in Firefox 33

Status

()

defect
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: cbook, Assigned: martijn.martijn)

Tracking

({intermittent-failure})

unspecified
mozilla34
ARM
Android
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(firefox32 wontfix, firefox33 fixed, firefox34 fixed, firefox-esr24 wontfix, firefox-esr31 wontfix)

Details

()

Attachments

(1 attachment)

Android 2.3 Emulator mozilla-inbound opt test mochitest-1 on 2014-08-25 18:08:33 PDT for push 0843c26857a9

slave: tst-linux64-spot-912

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



31 INFO TEST-UNEXPECTED-FAIL | /tests/content/base/test/csp/test_CSP_bug885433.html | Inline script should be allowed - Result logged after SimpleTest.finish()
This should fix it. This makes sure that the load call for checkBlocked is always called before the load call for checkAllowed.
Attachment #8479104 - Flags: review?(garrett.f.robinson+mozilla)
Assignee: nobody → martijn.martijn
Attachment #8479104 - Flags: review?(sstamm)
Comment on attachment 8479104 [details] [diff] [review]
1058451.diff (for check-in)

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

> This should fix it. This makes sure that the load call for checkBlocked 
> is always called before the load call for checkAllowed.

I think it's the other way around: checkAllowed runs and doesn't start the "blocked" test until after the first test completes.  Regardless, this serializes the two tests and doesn't call SimpleTest.finish() until after both run (in checkBlocked).  Thanks for the fix.

Garrett: I'm clearing your review flag because this is a pretty simple patch and I don't think it requires double-checking.  Feel free to take a look, though, if you want.
Attachment #8479104 - Flags: review?(sstamm)
Attachment #8479104 - Flags: review?(garrett.f.robinson+mozilla)
Attachment #8479104 - Flags: review+
(In reply to Sid Stamm [:geekboy or :sstamm] from comment #6)
> > This should fix it. This makes sure that the load call for checkBlocked 
> > is always called before the load call for checkAllowed.
> 
> I think it's the other way around: checkAllowed runs and doesn't start the

Yeah, sorry, I mixed it up.
Keywords: checkin-needed
Attachment #8479104 - Attachment description: 1058451.diff → 1058451.diff (for check-in)
https://hg.mozilla.org/mozilla-central/rev/c16359660b11
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
You need to log in before you can comment on or make changes to this bug.