If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

test_csp_redirects.html is busted

RESOLVED FIXED

Status

()

Core
DOM: Core & HTML
RESOLVED FIXED
8 years ago
5 years ago

People

(Reporter: bsterne, Assigned: bsterne)

Tracking

({intermittent-failure})

Trunk
intermittent-failure
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

8 years ago
27891 ERROR TEST-UNEXPECTED-FAIL | /tests/content/base/test/test_csp_redirects.html | img-src-redir test: http://example.com/tests/content/base/test/file_csp_redirects_resource.sjs?res=image&testid=img-src-redir - got true, expected false
27894 ERROR TEST-UNEXPECTED-FAIL | /tests/content/base/test/test_csp_redirects.html | style-src-redir test: http://example.com/tests/content/base/test/file_csp_redirects_resource.sjs?res=style&testid=style-src-redir - got true, expected false

The image-redirect handing code wasn't reviewed or checked in yet, hence that test failing.

Need to turn off that test for now.
(Assignee)

Comment 1

8 years ago
also forgot to land the style portion of the csp redirect handling stuff, hence the style test failure.  Patch fixing both forthcoming...
(Assignee)

Comment 2

8 years ago
Created attachment 441150 [details] [diff] [review]
fix
Whiteboard: [orange]
Comment hidden (Treeherder Robot)
(Assignee)

Comment 4

8 years ago
http://hg.mozilla.org/mozilla-central/rev/ec330fa3c73d
http://hg.mozilla.org/mozilla-central/rev/f82c45224823
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
(Assignee)

Comment 7

7 years ago
This was fixed by:
http://hg.mozilla.org/mozilla-central/rev/ec330fa3c73d
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Blocks: 438871
Comment hidden (Treeherder Robot)

Comment 9

7 years ago
This should be reopened if it's being starred again.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Holy wild morphing bugs, Batman!

That was an instance of bug 611810, I think saying that this bug which was about an unready tests and some broken checking-in is also about a jseng crash is a stretch.
Status: REOPENED → RESOLVED
Last Resolved: 7 years ago7 years ago
Resolution: --- → FIXED
Keywords: intermittent-failure
Whiteboard: [orange]
You need to log in before you can comment on or make changes to this bug.