Closed Bug 1214277 Opened 9 years ago Closed 9 years ago

Devtools "Disable cache" setting breaks interception in non-e10s mode

Categories

(Core :: DOM: Service Workers, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla44
Tracking Status
e10s - ---
firefox44 --- fixed

People

(Reporter: ehsan.akhgari, Unassigned)

References

Details

Attachments

(1 file)

STR: 1. Create a new profile, and run Firefox using it, and open a non-e10s window. 2. Go to <https://marcoscaceres.github.io/sw-workshop/finished/>. Disable your network connection. Reload. Everything works. 3. Open devtools, go to Settings, and check "Disable cache (when toolbox is open)". 4. Reload (still with the network connection disabled.) The interception doesn't work.
Attachment #8673372 - Flags: review?(mcmanus) → review+
Comment on attachment 8673372 [details] [diff] [review] Avoid bypassing opening a cache entry for possibly intercepted channels Review of attachment 8673372 [details] [diff] [review]: ----------------------------------------------------------------- Michael, do you mind taking a look at this too? I was thinking about this patch yesterday, specifically about what it could possibly break that I need to test, and I can't really say for sure, so I'd appreciate a second opinion as well. :-) Thanks!
Attachment #8673372 - Flags: review?(michal.novotny)
Ping?
I guess I'll land this, and address any comments post landing. It's better to land sooner to see if it breaks something.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
Comment on attachment 8673372 [details] [diff] [review] Avoid bypassing opening a cache entry for possibly intercepted channels Sorry for the delay. Looks good to me, but I'd like Honza to have a look at it too, because he knows this code best.
Flags: needinfo?(honzab.moz)
Attachment #8673372 - Flags: review?(michal.novotny) → review+
Comment on attachment 8673372 [details] [diff] [review] Avoid bypassing opening a cache entry for possibly intercepted channels Review of attachment 8673372 [details] [diff] [review]: ----------------------------------------------------------------- Josh should check on this. However, we want to move away from using the cache for the intercept feature soon. It shows to have drawbacks... like this one.
Attachment #8673372 - Flags: review?(josh)
Yeah, I agree! It would be nice if we can talk about what to do in Orlando.
Comment on attachment 8673372 [details] [diff] [review] Avoid bypassing opening a cache entry for possibly intercepted channels Review of attachment 8673372 [details] [diff] [review]: ----------------------------------------------------------------- Agreed. The more edge cases like these that we uncover, the stronger my desire to rewrite it all becomes.
Attachment #8673372 - Flags: review?(josh) → review+
Flags: needinfo?(honzab.moz)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: