Closed Bug 1005079 Opened 9 years ago Closed 9 years ago

[Perf] Move index check from CacheFile to CacheEntry to speed up

Categories

(Core :: Networking: Cache, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla32

People

(Reporter: mayhemer, Assigned: mayhemer)

References

Details

Attachments

(1 file)

The check whether we have an entry now happens in the CacheFile code and the answer is coming via a main thread loop.  We can avoid it when the check happens in CacheEntry (above).

Michal originally put this check to CacheEntry, but I wanted him from some reason I can't recall now put it to CacheFile.  Having this on the higher level is although better.

Found during examination of the jsreftests debug timeout regression.
Attached patch v1Splinter Review
Assignee: nobody → honzab.moz
Status: NEW → ASSIGNED
Attachment #8416550 - Flags: review?(michal.novotny)
Attachment #8416550 - Flags: review?(michal.novotny) → review+
https://hg.mozilla.org/mozilla-central/rev/0d4586d5273b
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla32
You need to log in before you can comment on or make changes to this bug.