2060912 - view-source re-requests rather than using the cached source

Status: NEW

(Core :: DOM: Navigation, defect)

Description

[Vincent Hilla [:vhilla]]

Attachment: server.py

Using the context menu to view page source re-requests the source rather than showing a cached version. If the request isn't idempotent, we might show different source than what is rendered.

Comment 1

[Vincent Hilla [:vhilla]]

See bug 55583, seems like this has been a bug previously. It does work in Chrome.

Claude says this was regressed by bug 1655572.

Cause according to Claude: nsDocShell::LoadPageAsViewSource fetches the viewed document's cache key and passes it to InternalLoad, but every SetCacheKey call in DoURILoad is gated on the load type (nsDocShell.cpp:9262-9288, requiring LOAD_HISTORY or LOAD_RELOAD_*), and the view-source load state defaults to LOAD_NORMAL. The cache key is the HTTP post id (nsHttpChannel::SetCacheKey → mPostID, nsHttpChannel.cpp:11311, appended to the cache entry id at :5118), so without it the channel generates a fresh post id, misses the cache, and goes to the network.

Comment 2

[BugBot [:suhaib / :marco]]

Set release status flags based on info from the regressing bug 1655572