Closed Bug 1719281 Opened 4 years ago Closed 4 years ago

QM: Add a generic wrapper for running initializations

Categories

(Core :: Storage: Quota Manager, task, P2)

task

Tracking

()

RESOLVED FIXED
92 Branch
Tracking Status
firefox92 --- fixed

People

(Reporter: janv, Assigned: janv)

References

Details

Attachments

(2 files)

We need a generic wrapper for running initializations which can handle the NS_ERROR_ABORT special case and correctly record first initialization attempts.
We currently handle NS_ERROR_ABORT in FirstInitializationAttempts::RecordFirstInitializationAttempt which is not suitable for that because the method is supposed to always set the recorded flag.

ExecuteInitialization now takes a function argument instead of nsresult so
initializations are executed in the context of ExecuteInitialization. This
patch also removes a temporary overload of ExecuteInitialization.

Depends on D119182

Pushed by jvarga@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/d6801824d47d QM: Add a generic wrapper for running initializations; r=dom-storage-reviewers,asuth https://hg.mozilla.org/integration/autoland/rev/58b39579def6 QM: Execute initializations in ExecuteInitialization() for real; r=dom-storage-reviewers,asuth
Pushed by jvarga@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/80941a3e3917 QM: Add a generic wrapper for running initializations; r=dom-storage-reviewers,asuth https://hg.mozilla.org/integration/autoland/rev/370f420a7bf8 QM: Execute initializations in ExecuteInitialization() for real; r=dom-storage-reviewers,asuth
Flags: needinfo?(jvarga)
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 92 Branch
Regressions: 1723503
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: