133884 - "Source" in Page Info lies (almost always says "Disk Cache")

Status: RESOLVED INVALID

(SeaMonkey :: Page Info, defect)

Description

[Jesse Ruderman]

Steps to reproduce:
1. Go to a random page you haven't been to before (without a ? in the URL).
2. Ctrl+I.

Result: Page Info says "Source: Disk Cache"
Expected: "Source: Web"

It would be nice if the sources included reasons:
 Source: Web (wasn't in cache)
 Source: Web (cached version had expired)
 Source: Web (cached version failed validation)
 Source: Cache (validated freshness through Web)
 Source: Cache (hasn't expired, didn't check for freshness)
 Source: Cache (expired, but in offline mode)

But the important thing is that Page Info shouldn't lie.

Comment 1

[Daniel Brooks [:db48x]]

it doesn't lie, the page really is in the cache. 

Comment 2

[Jesse Ruderman]

Is the page info entry supposed to say whether the page is cached, or whether
the page displayed came from the cache?

Comment 3

[Daniel Brooks [:db48x]]

both :)

if the cache (either memory or disk) is active, the page is downladed, stored in
the cache, and then displayed from the cache. Currently I don't think there is
an easy way to know if mozilla actually had to go out over the network and
download everything, or if it had everything in the cache already, which, I'll
admit, would be more useful than the current implementation. There might be away
to guess though. I'll check.

Comment 4

[Daniel Brooks [:db48x]]

no, there doesn't appear to be any way to guess. any ideas?

Comment 5

[Philip Chee]

The general tab doesn't show Source: in the current implementation.
Closing as INVALID.