Closed
Bug 975255
Opened 11 years ago
Closed 11 years ago
HTTP cache v2: optimize CacheIOThread::Target()
Categories
(Core :: Networking: Cache, defect)
Core
Networking: Cache
Tracking
()
RESOLVED
FIXED
mozilla30
People
(Reporter: mayhemer, Assigned: mayhemer)
References
Details
Attachments
(1 file)
|
677 bytes,
patch
|
michal
:
review+
|
Details | Diff | Splinter Review |
I realized we need to enter the monitor only to wait for the target to be published to the member (so very rarely soon after the startup). Otherwise, it's safe to just take it.
|
Assignee | |
Updated•11 years ago
|
OS: Windows 7 → All
Hardware: x86_64 → All
|
|
Assignee | |
Comment 1•11 years ago
|
||
This is called too often. Should be fixed for first version (patch trivial).
Severity: normal → major
|
|
Assignee | |
Comment 2•11 years ago
|
||
Assignee: nobody → honzab.moz
Status: NEW → ASSIGNED
Attachment #8379684 -
Flags: review?(michal.novotny)
|
||
Updated•11 years ago
|
Attachment #8379684 -
Flags: review?(michal.novotny) → review+
|
|
Assignee | |
Comment 3•11 years ago
|
||
|
|
Assignee | |
Comment 4•11 years ago
|
||
There is no direct test, but this is a generally used code tested a lot during any test run.
Flags: in-testsuite+
|
|
||
Comment 5•11 years ago
|
||
|
||
Comment 6•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla30
You need to log in
before you can comment on or make changes to this bug.
Description
•