Bug 1810018 - drop obsolete comments from various XUL components that no longer use DTDs, r=mconley

Differential Revision: https://phabricator.services.mozilla.com/D166731
This commit is contained in:
Gijs Kruitbosch 2023-01-17 14:44:33 +00:00
parent 70bdc6bc9b
commit 20a1ba5105
3 changed files with 2 additions and 23 deletions

View File

@ -233,10 +233,6 @@ this.DateTimeBoxWidget = class {
}
generateContent() {
/*
* Pass the markup through XML parser purely for the reason of loading the localization DTD.
* Remove it when migrate to Fluent (bug 1504363).
*/
const parser = new this.window.DOMParser();
let parserDoc = parser.parseFromString(
`<div class="datetimebox" xmlns="http://www.w3.org/1999/xhtml" role="none">

View File

@ -2805,10 +2805,6 @@ this.VideoControlsImplWidget = class {
}
generateContent() {
/*
* Pass the markup through XML parser purely for the reason of loading the localization DTD.
* Remove it when migrate to Fluent.
*/
const parser = new this.window.DOMParser();
let parserDoc = parser.parseFromString(
`<div class="videocontrols" xmlns="http://www.w3.org/1999/xhtml" role="none">
@ -3097,10 +3093,6 @@ this.NoControlsMobileImplWidget = class {
}
generateContent() {
/*
* Pass the markup through XML parser purely for the reason of loading the localization DTD.
* Remove it when migrate to Fluent.
*/
const parser = new this.window.DOMParser();
let parserDoc = parser.parseFromString(
`<div class="videocontrols" xmlns="http://www.w3.org/1999/xhtml" role="none">
@ -3149,10 +3141,6 @@ this.NoControlsPictureInPictureImplWidget = class {
}
generateContent() {
/*
* Pass the markup through XML parser purely for the reason of loading the localization DTD.
* Remove it when migrate to Fluent.
*/
const parser = new this.window.DOMParser();
let parserDoc = parser.parseFromString(
`<div class="videocontrols" xmlns="http://www.w3.org/1999/xhtml" role="none">
@ -3332,10 +3320,6 @@ this.NoControlsDesktopImplWidget = class {
}
generateContent() {
/*
* Pass the markup through XML parser purely for the reason of loading the localization DTD.
* Remove it when migrate to Fluent.
*/
const parser = new this.window.DOMParser();
let parserDoc = parser.parseFromString(
`<div class="videocontrols" xmlns="http://www.w3.org/1999/xhtml" role="none">

View File

@ -414,9 +414,8 @@
".wizard-header-label"
);
// First deal with fluent. Ideally, we'd stop supporting anything else,
// but right now the migration wizard still uses non-fluent l10n
// (fixing is bug 1518234), as do some comm-central consumers
// (bug 1627049). Removing the DTD support is bug 1627051.
// but some comm-central consumers still use DTDs. (bug 1627049).
// Removing the DTD support is bug 1627051.
if (this.currentPage.hasAttribute("data-header-label-id")) {
let id = this.currentPage.getAttribute("data-header-label-id");
document.l10n.setAttributes(labelElement, id);