[socket-proc] Land socket process without http part
Categories
(Core :: Networking, enhancement, P1)
Tracking
()
Tracking | Status | |
---|---|---|
firefox66 | --- | fixed |
People
(Reporter: kershaw, Assigned: kershaw)
References
(Blocks 1 open bug)
Details
(Whiteboard: [necko-triaged])
Attachments
(10 files)
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review |
This is about landing socket process without http part to mc.
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 1•5 years ago
|
||
Assignee | ||
Comment 2•5 years ago
|
||
This patch is quite straightforward. Just add socket process support.
Assignee | ||
Comment 3•5 years ago
|
||
To setup memory reporter on socket process, this patch modifies the PSocketProcess protocol to implement the same memory reporting functions as the PContent and PGPU protocols.
Assignee | ||
Comment 4•5 years ago
|
||
Assignee | ||
Comment 5•5 years ago
|
||
Assignee | ||
Comment 6•5 years ago
|
||
Assignee | ||
Comment 7•5 years ago
|
||
Assignee | ||
Updated•5 years ago
|
Updated•5 years ago
|
Updated•5 years ago
|
Updated•5 years ago
|
Assignee | ||
Comment 8•5 years ago
|
||
Comment 9•5 years ago
|
||
Assignee | ||
Comment 10•5 years ago
|
||
Assignee | ||
Comment 11•5 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=ef649d0ee3ffa70599128103165beddc723f293b Looks good when socket process is enabled.
Comment 12•5 years ago
|
||
Pushed by kjang@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/4e94bbb90315 P1: Start the new socket process basics (prefs, full xpcom init, logging, no sandboxing) r=mayhemer,dragana
Comment 13•5 years ago
|
||
Pushed by kjang@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/c900ac2519f5 P1.1: Create a reusable class to pass prefs to child processes r=dragana
Comment 14•5 years ago
|
||
Pushed by kjang@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/edbda5ee5fd5 P2: Setup crash reporter on socket process r=dragana,mayhemer
Comment 15•5 years ago
|
||
Pushed by kjang@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/a81f83df08d5 P3: Setup memory reporter on socket process r=dragana,mayhemer
Comment 16•5 years ago
|
||
Pushed by kjang@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/86032a14d26f P4: Update online/offline status to socket process r=dragana,mayhemer
Comment 17•5 years ago
|
||
Pushed by kjang@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/460bbf0849e1 P5: Create IPC between content process and socket process r=dragana,mayhemer
Comment 18•5 years ago
|
||
Pushed by kjang@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/f648b5f1a7c2 P6: Create Background between content process and socket process r=dragana,mayhemer
Comment 19•5 years ago
|
||
Pushed by kjang@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/de65c456aad6 P7: Setup Gecko profiler on the socket process r=dragana,mayhemer,mstange
Comment 20•5 years ago
|
||
Pushed by kjang@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/46411c5de3da P8: Report telemetry from socket process to parent process r=dragana,mayhemer,janerik
Comment 21•5 years ago
|
||
Pushed by kjang@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/56329b5f1844 P9: socket oepration should wait until socket process launch r=mayhemer,dragana,kershaw
Comment 22•5 years ago
|
||
Backed out for build bustages
Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=221325594&repo=autoland&lineNumber=16062
Android bustages push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception&classifiedState=unclassified&revision=edbda5ee5fd53f9ba0ab75477f08c7b29088eca1&selectedJob=221326328
Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=221326328&repo=autoland&lineNumber=29490
Backout: https://hg.mozilla.org/integration/autoland/rev/3f76ed638d83ade10f10cfecd2919936335a4aea
Assignee | ||
Comment 23•5 years ago
|
||
(In reply to Andreea Pavel [:apavel] from comment #22)
Backed out for build bustages
Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=221325594&repo=autoland&lineNumber=16062
Android bustages push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception&classifiedState=unclassified&revision=edbda5ee5fd53f9ba0ab75477f08c7b29088eca1&selectedJob=221326328
Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=221326328&repo=autoland&lineNumber=29490
Backout: https://hg.mozilla.org/integration/autoland/rev/3f76ed638d83ade10f10cfecd2919936335a4aea
Thanks for the back out.
I've fixed the build issue.
https://treeherder.mozilla.org/#/jobs?repo=try&revision=c37adc4de915898f4e12134c491898b25b7a03c2
Comment 24•5 years ago
|
||
Pushed by kjang@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/c083f11eb58d P1: Start the new socket process basics (prefs, full xpcom init, logging, no sandboxing) r=mayhemer,dragana
Comment 25•5 years ago
|
||
Pushed by kjang@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/696250a73120 P1.1: Create a reusable class to pass prefs to child processes r=dragana
Comment 26•5 years ago
|
||
Pushed by kjang@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/ba234cd89920 P2: Setup crash reporter on socket process r=dragana,mayhemer
Comment 27•5 years ago
|
||
Pushed by kjang@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/7c4eddc3a80d P3: Setup memory reporter on socket process r=dragana,mayhemer
Comment 28•5 years ago
|
||
Pushed by kjang@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/ca35e0cd0782 P4: Update online/offline status to socket process r=dragana,mayhemer
Comment 29•5 years ago
|
||
Pushed by kjang@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/b5904e4ffc0c P5: Create IPC between content process and socket process r=dragana,mayhemer
Comment 30•5 years ago
|
||
Pushed by kjang@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/5b73660fb915 P6: Create Background between content process and socket process r=dragana,mayhemer
Comment 31•5 years ago
|
||
Pushed by kjang@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/c65c63971388 P7: Setup Gecko profiler on the socket process r=dragana,mayhemer,mstange
Comment 32•5 years ago
|
||
Pushed by kjang@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/4af076789478 P8: Report telemetry from socket process to parent process r=dragana,mayhemer,janerik
Comment 33•5 years ago
|
||
Pushed by juhsu@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/d5da954d4dd2 P9: socket oepration should wait until socket process launch r=mayhemer,dragana,kershaw
Comment 34•5 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/c083f11eb58d
https://hg.mozilla.org/mozilla-central/rev/696250a73120
https://hg.mozilla.org/mozilla-central/rev/ba234cd89920
https://hg.mozilla.org/mozilla-central/rev/7c4eddc3a80d
https://hg.mozilla.org/mozilla-central/rev/ca35e0cd0782
https://hg.mozilla.org/mozilla-central/rev/b5904e4ffc0c
https://hg.mozilla.org/mozilla-central/rev/5b73660fb915
https://hg.mozilla.org/mozilla-central/rev/c65c63971388
https://hg.mozilla.org/mozilla-central/rev/4af076789478
https://hg.mozilla.org/mozilla-central/rev/d5da954d4dd2
Comment 35•5 years ago
|
||
Landing P1.1 seems to have introduced a perma failure:
kershaw, can you, please, take a look?
Comment 36•5 years ago
|
||
Pushed by bhackett@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/07acdca43a9b Disable failing web replay tests.
Comment 37•5 years ago
|
||
I just pushed 07acdca43a9b to disable the web replay tests until I'm able to update web replay so that it works with this change (later today, probably).
Going forward, I'd like to disable these tests when they start failing instead of backing out other changes, to avoid impeding development in the rest of Gecko. After bug 1517837 lands it will be a one line change to disable all of these tests.
Updated•5 years ago
|
Comment 38•5 years ago
|
||
bugherder |
Updated•5 years ago
|
Description
•