Closed
Bug 122499
Opened 23 years ago
Closed 23 years ago
about:cache thinks expiration time of 0 means "no expiration time"
Categories
(Core :: Networking: Cache, defect, P1)
Core
Networking: Cache
Tracking
()
RESOLVED
FIXED
mozilla0.9.9
People
(Reporter: darin.moz, Assigned: gordon)
Details
(Whiteboard: patch)
Attachments
(1 file)
|
1.31 KB,
patch
|
gagan
:
review+
darin.moz
:
superreview+
|
Details | Diff | Splinter Review |
about:cache thinks expiration time of 0 means "no expiration time"
see nsAboutCache.cpp:292
and nsAboutCacheEntry.cpp:440
Darin, this covers all the instances you were thinking of, right?
Looking for r/sr.
Whiteboard: patch
Comment on attachment 69567 [details] [diff] [review]
proposed patch to reflect 0xFFFFFFFF mean "never expires".
seems ok to me. You might still wanna add some comments in the code though. I
wonder if darin was also refering to adding a mechanism that allowed for 'never
expires' Is that all covered with 0xFFFFFFFF case? r=gagan
Attachment #69567 -
Flags: review+
|
Reporter | |
Comment 4•23 years ago
|
||
Comment on attachment 69567 [details] [diff] [review]
proposed patch to reflect 0xFFFFFFFF mean "never expires".
sr=darin
Attachment #69567 -
Flags: superreview+
Patch checked in. Marking FIXED.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•