Open Bug 1825234 Opened 2 years ago Updated 2 years ago

Drop sanitize dialog's async initialization and sizeToContent now that it's superfluous

Categories

(Toolkit :: Data Sanitization, task, P3)

Desktop
All
task

Tracking

()

Tracking Status
firefox113 --- affected

People

(Reporter: Gijs, Unassigned)

References

(Blocks 1 open bug)

Details

After bug 1819664, if we set l10n IDs from DOMContentLoaded or earlier, they'll block load.

So instead of this:

let warningDesc = document.getElementById("sanitizeEverythingWarning");
// Ensure we've translated and sized the warning.
await document.l10n.translateFragment(warningDesc);
let rootWin = window.browsingContext.topChromeWindow;
await rootWin.promiseDocumentFlushed(() => {});

we should be able to drop this and make init sync again, as long as we call it from DOMContentLoaded, not onload (which would be a good idea more generally anyway).

Severity: -- → N/A
Priority: -- → P3
You need to log in before you can comment on or make changes to this bug.