Closed
Bug 1379872
Opened 7 years ago
Closed 7 years ago
Initialize ProtocolProxyService early in startup
Categories
(Core :: Networking, enhancement)
Core
Networking
Tracking
()
RESOLVED
FIXED
mozilla57
Tracking | Status | |
---|---|---|
firefox57 | --- | fixed |
People
(Reporter: swu, Assigned: xeonchen)
Details
(Whiteboard: [necko-active][proxy])
Attachments
(1 file)
Follow up bug from https://bugzilla.mozilla.org/show_bug.cgi?id=1366133#c26
Reporter | ||
Comment 1•7 years ago
|
||
Gary, would you like to take a look at this?
Flags: needinfo?(xeonchen)
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → xeonchen
Flags: needinfo?(xeonchen)
Whiteboard: [proxy] → [necko-active][proxy]
Comment hidden (mozreview-request) |
Assignee | ||
Comment 3•7 years ago
|
||
Comment on attachment 8890805 [details] Bug 1379872 - early initialize ProtocolProxyService; this causes test failure, cancel review first.
Attachment #8890805 -
Flags: review?(daniel)
Comment hidden (mozreview-request) |
Comment 5•7 years ago
|
||
The r? is left on the patch itself, but given the more recent comment I'll hold off...
Assignee | ||
Comment 6•7 years ago
|
||
(In reply to Daniel Stenberg [:bagder] from comment #5) > The r? is left on the patch itself, but given the more recent comment I'll > hold off... I found the reason that breaks on try, and filed bug 1386168 for follow-up. Looks like it passes on try now, so please feel free to review :)
Comment 7•7 years ago
|
||
mozreview-review |
Comment on attachment 8890805 [details] Bug 1379872 - early initialize ProtocolProxyService; https://reviewboard.mozilla.org/r/162010/#review177914
Attachment #8890805 -
Flags: review?(daniel) → review+
Pushed by gachen@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/4251e7033a49 early initialize ProtocolProxyService; r=bagder
Comment 9•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/4251e7033a49
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox57:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
You need to log in
before you can comment on or make changes to this bug.
Description
•