This bug was filed from the Socorro interface and is report bp-89657161-7203-4169-81e7-904e72170122. ============================================================= This is a diagnostic assert triggering: https://hg.mozilla.org/releases/mozilla-aurora/annotate/a891cb2fa602/dom/cache/Context.cpp#l954 It seems a Context is being destroyed while it has a mNextContext. That next context is also then being destroyed. Must be a shutdown condition.

The crash reason actually says: MOZ_RELEASE_ASSERT(!mData) Which makes more sense.

I don't think this bug has any user observable behavior in release. It is triggering because we are trying to create a Cache API Manager/Context pair during shutdown. That causes us to take this failure path: https://dxr.mozilla.org/mozilla-central/source/dom/cache/Context.cpp#993 The ~Context() destructor, however, wants to ensure that we release our shared Data object on the target IO thread. Unfortunately that can't happen if the quota system is never opened. We avoid dooming the shared Data object here: https://dxr.mozilla.org/mozilla-central/source/dom/cache/Context.cpp#924 Therefore, lets just explicitly clear mData to signal that there is no shared Data to release. This maintains the destructor invariants.

Pushed by bkelly@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/a522c83358fa Clear Cache API shared data if Context fails to initialize. r=asuth

We only need to uplift this to aurora since its just a diagnostic assertion problem.

Comment on attachment 8829477 [details] [diff] [review] Clear Cache API shared data if Context fails to initialize. r=asuth Approval Request Comment [Feature/Bug causing the regression]: Cache API [User impact if declined]: Diagnostic assertions in aurora. Should not have any impact in release code. [Is this code covered by automated tests?]: Extensive tests, but they don't trigger this shutdown condition. [Has the fix been verified in Nightly?]: Its a low frequency race condition and difficult to verify. [Needs manual test from QE? If yes, steps to reproduce]: No [List of other uplifts needed for the feature/fix]: None [Is the change risky?]: Minimal risk [Why is the change risky/not risky?]: There is minimal risk of regressions because this simply clears some state if an operation is canceled due to shutdown. This is a relatively rare condition. [String changes made/needed]: None

Comment on attachment 8829477 [details] [diff] [review] Clear Cache API shared data if Context fails to initialize. r=asuth Fix a crash. Aurora53+.