561449 - test_csp_redirects.html is busted

Status: RESOLVED FIXED

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

Description

[Brandon Sterne (:bsterne)]

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.

Comment 1

[Brandon Sterne (:bsterne)]

also forgot to land the style portion of the csp redirect handling stuff, hence the style test failure.  Patch fixing both forthcoming...

Comment 2

[Brandon Sterne (:bsterne)]

Attachment: fix

Comment 3

[Legacy TBPL/Treeherder Robot]

[e-mail]
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1272058647.1272059935.6674.gz
Rev3 Fedora 12x64 mozilla-central debug test mochitests-1/5, Started 14:37, finished 14:59

s: talos-r3-fed64-015
27885 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
27888 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

Comment 4

[Brandon Sterne (:bsterne)]

http://hg.mozilla.org/mozilla-central/rev/ec330fa3c73d
http://hg.mozilla.org/mozilla-central/rev/f82c45224823

Comment 5

[Legacy TBPL/Treeherder Robot]

[e-mail]
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1272061359.1272062138.13703.gz
Rev3 Fedora 12x64 mozilla-central opt test mochitests-1/5, Started 15:22, finished 15:36

s: talos-r3-fed64-020
27936 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
27939 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

Comment 6

[Legacy TBPL/Treeherder Robot]

[e-mail]
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1272060420.1272061964.13209.gz
Linux mozilla-central debug test mochitests-1/5, Started 15:07, finished 15:33

s: moz2-linux-slave10
27962 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
27965 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

Comment 7

[Brandon Sterne (:bsterne)]

This was fixed by:
http://hg.mozilla.org/mozilla-central/rev/ec330fa3c73d

Comment 8

[Legacy TBPL/Treeherder Robot]

surkov.alexander%gmail.com
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1289590144.1289590336.4871.gz
Rev3 Fedora 12x64 mozilla-central opt test mochitests-1/5 on 2010/11/12 11:29:04

s: talos-r3-fed64-006
TEST-UNEXPECTED-FAIL | /tests/content/base/test/test_csp_redirects.html | Exited with code 1 during test run
PROCESS-CRASH | /tests/content/base/test/test_csp_redirects.html | application crashed (minidump found)
Thread 0 (crashed)

Comment 9

[Josh Matthews [:jdm]]

This should be reopened if it's being starred again.

Comment 10

[Phil Ringnalda (:philor)]

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.