Add RtcpMuxPolicy to PeerConnection.

Add RtcpMuxPolicy to PeerConnection according to https://tools.ietf.org/html/draft-ietf-rtcweb-jsep-09#section-4.1.1

BUG=webrtc:4611
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=197386

[pthatcher2, 2015-04-30 22:40:04]

pthatcher@chromium.org changed reviewers:
+ pfeldman@chromium.org

[pfeldman, 2015-05-16 00:06:25]

You want rtc reviews in the list.

[pfeldman, 2015-05-16 00:06:30]

pfeldman@chromium.org changed reviewers:
- pfeldman@chromium.org

[pthatcher2, 2015-05-20 19:54:38]

pthatcher@chromium.org changed reviewers:
+ jiayl@chromium.org, juberti@chromium.org, pfeldman@chromium.org

[juberti2, 2015-05-20 22:29:49]

lgtm, fwiw

[pthatcher2, 2015-05-22 04:17:09]

On 2015/05/20 22:29:49, juberti2 wrote:
> lgtm, fwiw

pfeldman?  Can you take a look now?

[pthatcher2, 2015-05-22 04:17:29]

On 2015/05/22 04:17:09, pthatcher2 wrote:
> On 2015/05/20 22:29:49, juberti2 wrote:
> > lgtm, fwiw
> 
> pfeldman?  Can you take a look now?

I mean, now that Justin has LGed it.

[pthatcher2, 2015-05-22 15:36:06]

pthatcher@chromium.org changed reviewers:
+ tommi@chromium.org

[tommi (sloooow) - chröme, 2015-05-22 15:46:11]

https://codereview.chromium.org/1116253004/diff/20001/Source/modules/mediastr...
File Source/modules/mediastream/RTCPeerConnection.cpp (right):

https://codereview.chromium.org/1116253004/diff/20001/Source/modules/mediastr...
Source/modules/mediastream/RTCPeerConnection.cpp:132: RTCRtcpMuxPolicy
rtcpMuxPolicy = RTCRtcpMuxPolicyRequire;
Shouldn't this be initialized to WebRTCRtcpMuxPolicyNegotiate?

[pthatcher2, 2015-05-22 15:55:27]

https://codereview.chromium.org/1116253004/diff/20001/Source/modules/mediastr...
File Source/modules/mediastream/RTCPeerConnection.cpp (right):

https://codereview.chromium.org/1116253004/diff/20001/Source/modules/mediastr...
Source/modules/mediastream/RTCPeerConnection.cpp:132: RTCRtcpMuxPolicy
rtcpMuxPolicy = RTCRtcpMuxPolicyRequire;
On 2015/05/22 15:46:10, tommi wrote:
> Shouldn't this be initialized to WebRTCRtcpMuxPolicyNegotiate?

Yes, thank you for noticing that.  That could have been bad (or good, depending
on how you look at it :).

[tommi (sloooow) - chröme, 2015-05-22 17:18:17]

lgtm

[pthatcher2, 2015-06-08 22:34:40]

pthatcher@chromium.org changed reviewers:
+ mkwst@chromium.org

[pthatcher2, 2015-06-08 22:34:40]

Mike West, 

I saw you reviewed a similar WebRTC CL
(https://codereview.chromium.org/1171553002/).  I've been waiting for a while on
this CL, but it looks like pfeldman@ is pretty busy.  Can you take a look?

[Mike West, 2015-06-12 07:35:17]

On 2015/06/08 at 22:34:40, pthatcher wrote:
> Mike West, 
> 
> I saw you reviewed a similar WebRTC CL
(https://codereview.chromium.org/1171553002/).  I've been waiting for a while on
this CL, but it looks like pfeldman@ is pretty busy.  Can you take a look?

platform/ LGTM. Sorry about the delay!

[pthatcher2, 2015-06-18 17:09:34]

The CQ bit was checked by pthatcher@chromium.org

[pthatcher2, 2015-06-18 17:09:36]

The patchset sent to the CQ was uploaded after l-g-t-m from juberti@chromium.org
Link to the patchset: https://codereview.chromium.org/1116253004/#ps40001
(title: "make negotiate the default")

[commit-bot: I haz the power, 2015-06-18 17:09:58]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1116253004/40001

[commit-bot: I haz the power, 2015-06-18 18:34:34]

Committed patchset #3 (id:40001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=197386