Open
Bug 1372961
Opened 8 years ago
Updated 2 years ago
Redundant set of "response-head" metadata on cache entries
Categories
(Core :: Networking: HTTP, enhancement, P3)
Core
Networking: HTTP
Tracking
()
NEW
People
(Reporter: mayhemer, Unassigned)
Details
(Whiteboard: [necko-backlog])
nsHttpChannel::ProcessNotModified() calls mCacheEntry->SetMetaDataElement("response-head") with flattened cached response head and just few lines below calls AddCacheEntryHeaders(..) for the same entry after the cached response head has been moved to response head. That again sets an unmodified "response-head" meta on the same entry.
We can probably remove this:
https://dxr.mozilla.org/mozilla-central/rev/b266a8d8fd595b84a7d6218d7b8c6b7af0b5027c/netwerk/protocol/http/nsHttpChannel.cpp#3382-3386
Comment 1•8 years ago
|
||
Moving to backlog. Please assign it to someone if this needs to be fixed soon.
Whiteboard: [necko-backlog]
Comment 2•7 years ago
|
||
Bulk change to priority: https://bugzilla.mozilla.org/show_bug.cgi?id=1399258
Priority: -- → P1
Comment 3•7 years ago
|
||
Bulk change to priority: https://bugzilla.mozilla.org/show_bug.cgi?id=1399258
Priority: P1 → P3
Updated•2 years ago
|
Severity: minor → S4
You need to log in
before you can comment on or make changes to this bug.
Description
•