Closed Bug 1778998 Opened 4 years ago Closed 3 years ago

Update host-utils after bug 1667455

Categories

(Core :: XPCOM, task)

task

Tracking

()

RESOLVED DUPLICATE of bug 1792805

People

(Reporter: arai, Unassigned)

References

Details

bug 1667455 exposes Services property on privileged global, and going to remove Services.jsm file.

the hostutils uses in-tree httpd.js, and the file needs to be compatible both with old/new binary.
in bug 1667455 patch, httpd.js is updated to be compatible both with patched/unpatched code.
https://hg.mozilla.org/integration/autoland/rev/911d540fce6f

-const { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");
+// httpd.js is loaded by Android hostutils that's not up to date.
+// Fallback to Services.jsm if `Services` global variable isn't yet available.
+const Services =
+  globalThis.Services ||
+  ChromeUtils.import("resource://gre/modules/Services.jsm").Services;

The hostutils needs to be updated after bug 1667455 patch gets merged, and then the backward compatible code in httpd.js can be removed.

Blocks: 1778999
No longer depends on: 1667455
Depends on: 1780695

Just spotted this, I wonder if it should depend on bug 1792805?

it looks like bug 1792805 is already getting handled?
if that's the case, this bug can be duped against it.

Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.