Closed
Bug 1436184
Opened 6 years ago
Closed 6 years ago
Remove definitions of Ci, Cr, Cc, and Cu from httpd.js and .sjs files
Categories
(Firefox :: General, enhancement, P3)
Firefox
General
Tracking
()
RESOLVED
FIXED
Firefox 60
| Tracking | Status | |
|---|---|---|
| firefox60 | --- | fixed |
People
(Reporter: mccr8, Assigned: mccr8)
References
Details
Attachments
(1 file)
httpd.js and .sjs files are run on the version of XPCShell in hostutils, so removing Ci etc from them is going to require updating hostutils. Once that is done, a patch can be generated by backout out commit 9deac2849858c676c1eddb25f783b33d94eaab8a from my moz-source-tools directory ( https://github.com/amccreight/moz-source-tools ) and running the decomponents.py script.
| Comment hidden (mozreview-request) |
|
||
Updated•6 years ago
|
Priority: -- → P3
|
Assignee | |
Comment 2•6 years ago
|
||
This is green with the patch in the blocking bug: https://treeherder.mozilla.org/#/jobs?repo=try&revision=68783190ba22298f6db1be507109d203bcc62534
| Comment hidden (mozreview-request) |
|
|
Assignee | |
Comment 4•6 years ago
|
||
These changes should all be part of the patch you reviewed already, so it probably doesn't need careful review.
|
||
Comment 5•6 years ago
|
||
| mozreview-review | ||
Comment on attachment 8948834 [details] Bug 1436184 - Remove definitions of Ci, Cr, Cc, and Cu from httpd.js and .sjs files. https://reviewboard.mozilla.org/r/218206/#review224624
Attachment #8948834 -
Flags: review?(florian) → review+
Pushed by amccreight@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/dce5f1c884cc Remove definitions of Ci, Cr, Cc, and Cu from httpd.js and .sjs files. r=florian
|
||
Comment 7•6 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/dce5f1c884cc
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox60:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 60
You need to log in
before you can comment on or make changes to this bug.
Description
•