1651235 - Enable WebRTC usage of socket process on early beta and earlier

Status: VERIFIED FIXED

(Core :: Networking, enhancement, P2)

Description

[Michael Froman [:mjf]]

We're ready to enable webrtc on the socket process in early beta and earlier (rather than just nightly).

Comment 1

[Michael Froman [:mjf]]

Attachment: Bug 1651235 - enable webrtc usage of socket process on early beta and earlier. r?drno!

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/42aa9267e077
enable webrtc usage of socket process on early beta and earlier. r=drno

Comment 3

[Alexandru Michis [:malexandru]]

https://hg.mozilla.org/mozilla-central/rev/42aa9267e077

Comment 4

[Michael Froman [:mjf]]

Comment on attachment 9162016 [details]
Bug 1651235 - enable webrtc usage of socket process on early beta and earlier. r?drno!

Beta/Release Uplift Approval Request

• User impact if declined: This is to allow QE to qualify the socket process sandbox in Beta.
• Is this code covered by automated tests?: Yes
• Has the fix been verified in Nightly?: Yes
• Needs manual test from QE?: Yes
• If yes, steps to reproduce: This is general sandbox qualifications for webrtc using the socket process. QE should follow the same process used to qualify the socket process sandbox in Nightly.
• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): This is only a pref change to enable webrtc usage of the socket process in early beta. It has been used in Nightly for quite some time now.
• String changes made/needed:

Comment 5

[Ryan VanderMeulen [:RyanVM]]

We only have one early beta build left this cycle - is this really worth taking on Beta79 at this point vs. letting it ride to Beta80 naturally instead?

Comment 6

[Michael Froman [:mjf]]

That is up to Haik and/or Nils more than me.

Comment 7

[Haik Aftandilian [:haik]]

Nils is driving the enabling of the socket process so I'll let him respond on this.

Comment 8

[Nils Ohlmeier [:drno]]

As QA managed to test the feature in 79 by manually flipping thee pref we do have green lights to use it in 79. So I don't think that we need the uplift any more.

Comment 9

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 9162016 [details]
Bug 1651235 - enable webrtc usage of socket process on early beta and earlier. r?drno!

Thanks everyone.

Comment 10

[Brindusa Tot, Desktop QA]

Verified as fixed, the feature webrtc on the socket process is enabled (by default: “network.process.enabled” =true and “media.peerconnection.mtransport_process” =true) on early beta: Beta 80.0b1 - Beta 80.0b5 and Nightly 81 or earlier and disabled starting with Beta 80.0b6.