Closed
Bug 1789201
Opened 2 years ago
Closed 2 years ago
Expose Services to all DevTools modules
Categories
(DevTools :: General, task)
DevTools
General
Tracking
(firefox106 fixed)
RESOLVED
FIXED
106 Branch
| Tracking | Status | |
|---|---|---|
| firefox106 | --- | fixed |
People
(Reporter: ochameau, Assigned: ochameau)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
In order to help the migration of the DevTools codebase to ES Modules (bug 1525652), we should be exposing Services to all modules.
This symbol is being made available to all privileged ES Modules.
|
Assignee | |
Comment 1•2 years ago
|
||
This will help transition to ES Modules as this symbol is exposed to them.
$ sed -ie "/require(.Services.)/d" $(git grep -l 'require("Services")' devtools/)
$ sed -ie "/loader.lazyRequireGetter(this, .Services./d" $(git grep -l 'loader.lazyRequireGetter(this, "Services"' devtools/)
- the edition of builtin-modules.js + eslintrc.js
- manual eslint fixes
- removal of devtools-services in the debugger, except for jest tests
|
||
Updated•2 years ago
|
Assignee: nobody → poirot.alex
Status: NEW → ASSIGNED
Pushed by apoirot@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/32517628703d [devtools] Expose Services as a global to all DevTools modules. r=perftest-reviewers,nchevobbe,julienw,AlexandruIonescu
|
||
Comment 3•2 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
status-firefox106:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 106 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•