Bug 144386 – Main resource loaded via 304 response becomes empty if reloaded by user

• Status: RESOLVED
• Resolution: FIXED
• Component: Layout and Rendering

[Antti Koivisto]

Main resource loaded via 304 response becomes empty if reloaded by user.

[Antti Koivisto]

<rdar://problem/18480199>

[Antti Koivisto]

Created attachment 251953 [details]
patch

[Darin Adler]

Comment on attachment 251953 [details]
patch

I’ve wondered about this kind of thing for years. Part of the general “what do we want to keep when we reload and what needs to start fresh” topic for requests in general. Also related is the scheme for making reloads bypass cache appropriately.

Is this needed anywhere else? Does it only affect main resources? Is there some way to tie it more closely to other ways we have to change the request in a reloading situation?

[Antti Koivisto]

> Is this needed anywhere else? Does it only affect main resources? Is there
> some way to tie it more closely to other ways we have to change the request
> in a reloading situation?

Main document loading is a rare case where we have a long-lived ResourceRequest not owned by a CachedResource. The refactoring that moved us from entirely special main resource loading to CachedResources based loading (using CachedRawResource) is basically incomplete so we end up with weird special handling.

[Antti Koivisto]

https://trac.webkit.org/r183555

[Alexey Proskuryakov]

http/tests/cache/main-resource-304-reload.html fails on Windows:

https://webkit-test-results.appspot.com/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=http%2Ftests%2Fcache%2Fmain-resource-304-reload.html

[Antti Koivisto]

Marked it failing on Windows in http://trac.webkit.org/changeset/183573 (like most of the cache test).