Closed Bug 1614610 Opened 4 years ago Closed 4 years ago

Have cache occupancy for each content-type in cache index

Categories

(Core :: Networking: Cache, task, P1)

task

Tracking

()

RESOLVED FIXED
mozilla75
Tracking Status
firefox75 --- fixed

People

(Reporter: michal, Assigned: michal)

References

Details

(Whiteboard: [necko-triaged])

Attachments

(2 files)

Right not we calculate the size for each content type when doing a telemetry report (https://searchfox.org/mozilla-central/rev/a1592902acabf9bded973067133baaac1457f3d3/netwerk/cache2/CacheIndex.cpp#3920), but for bug 1548544 we need to know the size at any moment, so we need to add it to CacheIndexStats (https://searchfox.org/mozilla-central/rev/a1592902acabf9bded973067133baaac1457f3d3/netwerk/cache2/CacheIndex.h#459).

Pushed by mnovotny@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0a56cd6537a0
Have cache occupancy for each content-type in cache index r=valentin
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla75

Initialization was missing in CacheIndexStats::CacheIndexStats() and CacheIndexStats::Clear(). Comparison was also added to CacheIndexStats::operator==().

Pushed by mnovotny@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/3a36f52d40bb
Fix missing initialization of CacheIndexStats::mCountByType[] and CacheIndexStats::mSizeByType[] r=valentin
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: