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)

enhancement

Tracking

()

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
Moving to backlog. Please assign it to someone if this needs to be fixed soon.
Whiteboard: [necko-backlog]
Priority: -- → P1
Priority: P1 → P3
Severity: minor → S4
You need to log in before you can comment on or make changes to this bug.