956331 - HTTP cache v2: Leaks with patch for bug 951713

Status: RESOLVED INVALID

(Core :: Networking: Cache, defect)

Description

[Honza Bambas (:mayhemer)]

According try run at [1] it seems like bug 951713 causes leaks.  It is the only change between current and previous m-c changesets I've merged gum to.  Previous merge didn't suffer from this leak.  I've never pushed a try run for that bug.  The run at [1] is just m-c at [2] - having the bug 951713 patch - with enabling the cache2 code.


[1] https://tbpl.mozilla.org/?tree=Try&rev=9e5b030a35c1
[2] http://hg.mozilla.org/mozilla-central/rev/5c7fa2bfea8b

Comment 1

[Honza Bambas (:mayhemer)]

https://tbpl.mozilla.org/?tree=Try&rev=74512f707010 (m-c with cache2=on)
https://tbpl.mozilla.org/?tree=Try&rev=fe9666dabb11 (m-c, backout of 951713, cache2=on)

Comment 2

[Honza Bambas (:mayhemer)]

Rerun shows the leaks are there regardless the patch for bug 951713.  Maybe just raises the frequency.