Closed
Bug 1479236
Opened 7 years ago
Closed 7 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•7 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•7 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•7 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox63:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 63
|
||
Comment 5•7 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•7 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
•