Closed Bug 1213150 Opened 6 years ago Closed 6 years ago

Some nsIInterceptedChannel cleanup

Categories

(Core :: DOM: Service Workers, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla44
Tracking Status
firefox43 --- wontfix
firefox44 --- fixed

People

(Reporter: ehsan.akhgari, Assigned: ehsan.akhgari)

References

Details

Attachments

(3 files)

This doesn't need to block v1 but I'm nominating it for easier tracking of the uplifts.  I have some cleanup patches.
Status: NEW → ASSIGNED
Attachment #8671713 - Flags: review?(josh) → review+
Attachment #8671714 - Flags: review?(josh) → review+
Attachment #8671715 - Flags: review?(josh) → review+
Comment on attachment 8671713 [details] [diff] [review]
Part 1: Add a nsContentUtils::IsNonSubresourceRequest helper

These are cleanup patches that would be nice to backport to Aurora for easier uplift of service worker changes.  They should not affect anything but service workers.
Attachment #8671713 - Flags: approval-mozilla-aurora?
Comment on attachment 8671713 [details] [diff] [review]
Part 1: Add a nsContentUtils::IsNonSubresourceRequest helper

Please uplift to aurora. Part of service worker feature.
Attachment #8671713 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Ehsan, all the patches here or just Part 1?
Flags: needinfo?(ehsan)
All.
Flags: needinfo?(ehsan)
I'm hitting merge conflicts trying to uplift these. Ehsan, can you post rebased patches (and fold in that followup, I suppose)?
Flags: needinfo?(ehsan)
Let me uplift myself.  Thanks for trying!
Flags: needinfo?(ehsan)
We decided to not ship SW in 43.
Flags: needinfo?(ehsan)
You need to log in before you can comment on or make changes to this bug.