1476750 - Rename the preference media.webrtc.rsdparsa_enabled

Status: RESOLVED FIXED

(Core :: WebRTC: Signaling, enhancement, P2)

Description

[Johannes Willbold]

Rename the preference "media.webrtc.rsdparsa_enabled" to "media.peerconnection.rsdparsa.enabled".

Comment 1

[Johannes Willbold]

media.peerconnection.rsdpara.comapre

Comment 2

[Nils Ohlmeier [:drno]]

Lets actually use "media.peerconnection.sdp.rust.enabeld" and "media.peerconnection.sdp.rust.compare"

Comment 3

[Johannes Willbold]

Attachment: Bug 1476750: Updated preferences controlling Rust SDP Parse behavior,

Renamed the pref media.webrtc.rsdparsa_enabled to media.webrtc.rsdparsa_enabled
Added the pref media.peerconnection.rsdparsa.enabled
Added both to all.js

Review commit: https://reviewboard.mozilla.org/r/260112/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/260112/

Comment 4

[Dan Minor [:dminor]]

Comment on attachment 8995744 [details]
Bug 1476750: Updated preferences controlling Rust SDP Parse behavior,

https://reviewboard.mozilla.org/r/260112/#review267246

lgtm

::: commit-message-0e816:1
(Diff revision 1)
> +Bug 1476750: Renamed the preference media.webrtc.rsdparsa_enabled, r=dminor

The summary here should be something like "Updated preferences controlling Rust SDP Parse behavior" since you've changed more than one preference in this bug.

::: commit-message-0e816:4
(Diff revision 1)
> +Bug 1476750: Renamed the preference media.webrtc.rsdparsa_enabled, r=dminor
> +
> +Renamed the pref media.webrtc.rsdparsa_enabled to media.webrtc.rsdparsa_enabled
> +Added the pref media.peerconnection.rsdparsa.enabled

This should be: media.peerconnection.sdp.rust.compare

Please also mention that you're using it to control whether or not we compare rust and sdparsa results.

Comment 5

[Johannes Willbold]

Comment on attachment 8995744 [details]
Bug 1476750: Updated preferences controlling Rust SDP Parse behavior,

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/260112/diff/1-2/

Comment 6

[Nils Ohlmeier [:drno]]

Comment on attachment 8995744 [details]
Bug 1476750: Updated preferences controlling Rust SDP Parse behavior,

https://reviewboard.mozilla.org/r/260112/#review267370

Two more small fixes please.

::: media/webrtc/signaling/src/jsep/JsepSessionImpl.cpp:71
(Diff revision 2)
>    NS_ENSURE_SUCCESS(rv, rv);
>  
>    SetupDefaultCodecs();
>    SetupDefaultRtpExtensions();
>  
> -  mRunRustParser = Preferences::GetBool("media.webrtc.rsdparsa_enabled", false);
> +  mRunRustParser = Preferences::GetBool("media.peerconnection.rsdparsa.enabled",

I meant to use "sdp.rust" consistently for all prefs instead of "rsdparsa".

::: modules/libpref/init/all.js:424
(Diff revision 2)
>  pref("media.navigator.video.use_tmmbr", false);
>  pref("media.navigator.audio.use_fec", true);
>  pref("media.navigator.video.red_ulpfec_enabled", false);
>  
>  pref("media.peerconnection.dtmf.enabled", true);
> +pref("media.peerconnection.rsdparsa.enabled", false);

Replace "rsdparsa" with "sdp.rust" here as well.

Comment 7

[Johannes Willbold]

Comment on attachment 8995744 [details]
Bug 1476750: Updated preferences controlling Rust SDP Parse behavior,

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/260112/diff/2-3/

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b57c92c69d8b
Updated preferences controlling Rust SDP Parse behavior, r=dminor,drno

Comment 9

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/b57c92c69d8b