Closed Bug 1170937 Opened 7 years ago Closed 7 years ago

Re-enable test_fetch_basic_http_sw_reroute.html and test_fetch_cors_sw_reroute.html test once we have fully support for redirects

Categories

(Core :: DOM: Service Workers, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla41
Tracking Status
firefox41 --- fixed

People

(Reporter: jaoo, Assigned: ehsan.akhgari)

References

Details

Attachments

(1 file)

See bug 1122161 comment 17.

(In reply to :Ehsan Akhgari (not reading bugmail, needinfo? me!) from comment #17)
> Bug 1164397 may also help.  Please file a new bug and make it depend on both
> this one and the other one, and then we can look into re-enabling those
> tests there.  Thanks!
Component: DOM → DOM: Service Workers
Depends on: 1164397
These tests are still failing...  I will investigate.
Assignee: nobody → ehsan
The bug here is simple, we always set the URL member of a response object from the corresponding request object, and we disregard the final URL of the channel after a redirect.  I have a fix.
Attachment #8617515 - Flags: review?(amarchesini) → review+
Depends on: 1173163
Relanded with the tests disabled in e10s or now.
Flags: needinfo?(ehsan)
https://hg.mozilla.org/mozilla-central/rev/90b8bf917db0
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla41
You need to log in before you can comment on or make changes to this bug.