Closed Bug 1513057 Opened 1 year ago Closed 1 year ago

[socket-proc] Land socket process without http part

Categories

(Core :: Networking, enhancement, P1)

enhancement

Tracking

()

RESOLVED FIXED
mozilla66
Fission Milestone M1
Tracking Status
firefox66 --- fixed

People

(Reporter: kershaw, Assigned: kershaw)

References

(Depends on 1 open bug, Blocks 2 open bugs)

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: nobody → kershaw
Priority: -- → P1
Whiteboard: [necko-triaged]
This patch is quite straightforward. Just add socket process support.
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.
Attachment #9030951 - Attachment description: Bug 1513057 - P6: Setup Gecko profiler on the socket process → Bug 1513057 - P7: Setup Gecko profiler on the socket process
Attachment #9030951 - Attachment description: Bug 1513057 - P7: Setup Gecko profiler on the socket process → Bug 1513057 - P67 Setup Gecko profiler on the socket process
Attachment #9030951 - Attachment description: Bug 1513057 - P67 Setup Gecko profiler on the socket process → Bug 1513057 - P7 Setup Gecko profiler on the socket process
Attachment #9030951 - Attachment description: Bug 1513057 - P7 Setup Gecko profiler on the socket process → Bug 1513057 - P7: Setup Gecko profiler on the socket process
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
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
Pushed by kjang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/edbda5ee5fd5
P2: Setup crash reporter on socket process r=dragana,mayhemer
Pushed by kjang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a81f83df08d5
P3: Setup memory reporter on socket process r=dragana,mayhemer
Pushed by kjang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/86032a14d26f
P4: Update online/offline status to socket process r=dragana,mayhemer
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
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
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
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
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
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
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
Pushed by kjang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ba234cd89920
P2: Setup crash reporter on socket process r=dragana,mayhemer
Pushed by kjang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7c4eddc3a80d
P3: Setup memory reporter on socket process r=dragana,mayhemer
Pushed by kjang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ca35e0cd0782
P4: Update online/offline status to socket process r=dragana,mayhemer
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
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
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
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
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

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.

Flags: needinfo?(kershaw)
Depends on: 1519728
Fission Milestone: --- → M1
Depends on: 1540289
You need to log in before you can comment on or make changes to this bug.