Closed
Bug 1438428
Opened 7 years ago
Closed 7 years ago
Change const {classes: CoC, interfaces: CoI, results: CoR, utils: CoU} = Components; to use Cc, Ci, Cr, and Cu globals in updates.js
Categories
(Toolkit :: Application Update, enhancement)
Toolkit
Application Update
Tracking
()
RESOLVED
FIXED
mozilla60
Tracking | Status | |
---|---|---|
firefox60 | --- | fixed |
People
(Reporter: robert.strong.bugs, Assigned: robert.strong.bugs)
Details
Attachments
(1 file)
25.92 KB,
patch
|
molly
:
review+
|
Details | Diff | Splinter Review |
This is no longer necessary with the introduction of the new globals
![]() |
Assignee | |
Comment 1•7 years ago
|
||
A couple of additional changes
Removed the code for app.update.test.loop which hasn't been used in years
Removed the no longer needed Cc, Ci, etc. constants from the xpcshell and mochitest-chrome common test files.
Attachment #8951162 -
Flags: review?(mhowell)
![]() |
Assignee | |
Comment 2•7 years ago
|
||
Pushed to try with a couple of other patches
https://treeherder.mozilla.org/#/jobs?repo=try&revision=3a2ed5feada5749918a9c433fccb2d6501a76bdf
Updated•7 years ago
|
Attachment #8951162 -
Flags: review?(mhowell) → review+
Pushed by rstrong@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/f8e7b9e6ab88
Use Cc, Ci, Cr, and Cu globals in updates.js. r=mhowell
Comment 4•7 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox60:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
You need to log in
before you can comment on or make changes to this bug.
Description
•