corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 1959758 - Don't depend on MozAfterPaint timing in test_set_attributes.html. r=Gijs

Browse Source 
I'm doing changes in the area and this test really doesn't need it.

Differential Revision: https://phabricator.services.mozilla.com/D245098
This commit is contained in:
Emilio Cobos Álvarez
2025-04-10 20:30:46 +00:00
parent 38b476436b
commit 62d253d96e
1 changed files with 10 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
dom/l10n/tests/mochitest/l10n_mutations/test_set_attributes.html
View File

@@ -17,16 +17,19 @@
let elem2 = document.getElementById("elem2"); let elem2 = document.getElementById("elem2");
is(elem.textContent.length, 0); is(elem.textContent.length, 0);
is(elem2.textContent.includes("Initial string"), true); is(elem2.textContent.includes("Initial string"), true);
let promiseMutations = new Promise(r => {
document.addEventListener("L10nMutationsFinished", r, { once: true });
});
document.l10n.setAttributes(elem, "crash-reports-title"); document.l10n.setAttributes(elem, "crash-reports-title");
elem2.setAttribute("data-l10n-args", JSON.stringify({bug: "New string"})); elem2.setAttribute("data-l10n-args", JSON.stringify({bug: "New string"}));
let verifyL10n = () => { await promiseMutations;
if (elem.textContent.length && elem2.textContent.includes("New string")) {
window.removeEventListener("MozAfterPaint", verifyL10n); isnot(elem.textContent.length, 0);
SimpleTest.finish(); ok(elem2.textContent.includes("New string"), "Should include new string");
} SimpleTest.finish();
};
window.addEventListener("MozAfterPaint", verifyL10n);
}); });
</script> </script>
</head> </head>