Summary: | Network Cache: do not use disk cache for Fetch media loads | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Olivier Blin <olivier.blin> | ||||
Component: | Media | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | NEW --- | ||||||
Severity: | Normal | CC: | achristensen, cdumez, cgarcia, darin, eocanha, ews-watchlist, jer.noble, koivisto, loic.yhuel, mcatanzaro, pnormand, webkit-bug-importer, youennf, zdobersek | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
See Also: |
https://bugs.webkit.org/show_bug.cgi?id=144259 https://bugs.webkit.org/show_bug.cgi?id=157405 https://bugs.webkit.org/show_bug.cgi?id=170676 https://bugs.webkit.org/show_bug.cgi?id=187544 https://bugs.webkit.org/show_bug.cgi?id=229813 |
||||||
Attachments: |
|
Description
Olivier Blin
2022-01-24 12:39:39 PST
Created attachment 449846 [details]
Patch
Comment on attachment 449846 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=449846&action=review Looks good > Source/WebKit/ChangeLog:23 > + No new tests (OOPS!). Can’t land a patch that still has this in it. Need to remove this line, and replace it with an explanation of why we are landing this patch without any tests. Might want to look back at the original XHR change to see how they wrote the test. (In reply to Olivier Blin from comment #0) > Should we attempt to factorize these checks? Yes. Pull request: https://github.com/WebKit/WebKit/pull/24243 |