Closed
Bug 1479236
Opened 6 years ago
Closed 6 years ago
Stop loading NetUtil.jsm in the content process
Categories
(Firefox :: General, enhancement)
Tracking
()
RESOLVED
FIXED
Firefox 63
Tracking | Status | |
---|---|---|
firefox63 | --- | fixed |
People
(Reporter: kmag, Assigned: kmag)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
NetUtil.jsm is large, and generally not especially useful. We should not be loading it in the content process.
Comment hidden (mozreview-request) |
Comment 2•6 years ago
|
||
mozreview-review |
Comment on attachment 8995761 [details] Bug 1479236: Stop loading NetUtil.jsm in the content process. https://reviewboard.mozilla.org/r/260124/#review267146
Attachment #8995761 -
Flags: review?(dtownsend) → review+
Assignee | ||
Comment 3•6 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/0462ba95cb8464436526a35551170f3252451c22 Bug 1479236: Stop loading NetUtil.jsm in the content process. r=Mossop
Comment 4•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/0462ba95cb84
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox63:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 63
Comment 5•6 years ago
|
||
(In reply to Kris Maglione [:kmag] from comment #0) > NetUtil.jsm is large, and generally not especially useful. We should not be > loading it in the content process. If we should not load it in the content process, it would be good to remove it from the whitelist at https://searchfox.org/mozilla-central/rev/033d45ca70ff32acf04286244644d19308c359d5/browser/base/content/test/performance/browser_startup_content.js#37 And it seems the test still thinks NetUtil.jsm is being loaded in content processes, or it would report an unused whitelist entry.
Assignee | ||
Comment 6•6 years ago
|
||
(In reply to Florian Quèze [:florian] from comment #5) > (In reply to Kris Maglione [:kmag] from comment #0) > > NetUtil.jsm is large, and generally not especially useful. We should not be > > loading it in the content process. > > If we should not load it in the content process, it would be good to remove > it from the whitelist at > https://searchfox.org/mozilla-central/rev/ > 033d45ca70ff32acf04286244644d19308c359d5/browser/base/content/test/ > performance/browser_startup_content.js#37 > > And it seems the test still thinks NetUtil.jsm is being loaded in content > processes, or it would report an unused whitelist entry. See bug 1479235.
You need to log in
before you can comment on or make changes to this bug.
Description
•