Closed Bug 1012331 Opened 10 years ago Closed 10 years ago

HTTP cache v2: Let CacheFile::PreloadChunks use mPreloadChunkCount instead of prefs

Categories

(Core :: Networking: Cache, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla32

People

(Reporter: mayhemer, Assigned: mayhemer)

References

Details

Attachments

(1 file)

Attached patch v1Splinter Review
To make the patch for bug 1010783 even more according Michal's requirements, CacheFile::PreloadChunks should also be limited with the stored mPreloadChunkCount, otherwise the Avail() == Read() promise may not be fulfilled when someone plays with the preference.

Very low priority since the pref is not even added to all.js (bug 1012327).
Attachment #8424440 - Flags: review?(michal.novotny)
Assignee: nobody → honzab.moz
Status: NEW → ASSIGNED
Attachment #8424440 - Flags: review?(michal.novotny) → review+
https://hg.mozilla.org/mozilla-central/rev/728ad0b94d8d
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla32
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: