Closed Bug 1011487 Opened 9 years ago Closed 9 years ago

HTTP cache v2: WalkCacheRunnable may release it's callback on a non-main thread

Categories

(Core :: Networking: Cache, defect)

defect
Not set
critical

Tracking

()

RESOLVED FIXED
mozilla32

People

(Reporter: mayhemer, Assigned: mayhemer)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

What causes and assertion failure in nsXPCWrappedJS::Release.
Attached patch v1Splinter Review
Simply proxy release.

https://tbpl.mozilla.org/?tree=Try&rev=141b3dec7e27
Assignee: nobody → honzab.moz
Status: NEW → ASSIGNED
Attachment #8423855 - Flags: review?(michal.novotny)
Attachment #8423855 - Flags: review?(michal.novotny) → review+
This patch had the misfortune of landing in the middle of a large bustage pileup on inbound that led to me having to revert to a last-good cset in lieu of waiting 3-4 hours for green instead. Unfortunately, that means this was backed out along with the others. If this was confirmed green on Try, it can re-land at your convenience.

https://hg.mozilla.org/integration/mozilla-inbound/rev/eb2a6f7785a2
https://hg.mozilla.org/mozilla-central/rev/020e705e89ba
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.