1814731 - Suspended channel blocks second channel to same URL waiting for cache entry

Status: NEW

(Core :: Networking: Cache, defect, P1)

Description

[Valentin Gosu [:valentin] (he/him)]

Kershaw's investigation in bug 1807262 seems accurate. I am fairly sure this is causing bug 1813132 as well.
The problem is that we're creating a channel (the early hint channel) then suspending it. Any later requests for the same URL will be waiting for the previous load to complete and write into the cache entry, but that channel is suspended, so they stall forever.
I've already confirmed the issue with a unit test, now I'm working on a fix.

I think the best approach here would be to doom the cache entry when suspending. I will put this behind a pref in case it causes any regressions.

Comment 1

[Valentin Gosu [:valentin] (he/him)]

Attachment: WIP: Bug 1814731 - Test that suspended HTTP channels don't block other channels using same cache entry r=#necko

Comment 2

[Valentin Gosu [:valentin] (he/him)]

Attachment: WIP: Bug 1814731 - Close cache on suspend r=#necko

Depends on D168789