Make test_bug369787.js not hang when the test fails

RESOLVED FIXED

Status

()

Core
Networking: HTTP
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: Biesinger, Assigned: Biesinger)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Created attachment 493146 [details] [diff] [review]
patch

If change_content_type throws an exception in test_bug369787.js, do_test_finished is never called and the test hangs.

Still trying to figure out why the test fails for me but not tinderbox, but meanwhile the attached patch fixes the hang when that happens.
Attachment #493146 - Attachment is patch: true
Attachment #493146 - Attachment mime type: application/octet-stream → text/plain
Attachment #493146 - Flags: review?(bzbarsky)
Assignee: nobody → cbiesinger
Status: NEW → ASSIGNED
Created attachment 493152 [details] [diff] [review]
patch v2

Found the actual reason for the failure - I have a systemwide PAC file set. onStartRequest should set channel to request. This patch does that (in addition to the try..catch from the previous version).
Attachment #493146 - Attachment is obsolete: true
Attachment #493152 - Flags: review?(bzbarsky)
Attachment #493146 - Flags: review?(bzbarsky)
Comment on attachment 493152 [details] [diff] [review]
patch v2

r=me
Attachment #493152 - Flags: review?(bzbarsky) → review+
http://hg.mozilla.org/mozilla-central/rev/1cb9c0d5b701
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.