Refactor WebRtcVideoReceiveStream::FilterSupportedCodecs

Refactor WebRtcVideoReceiveStream::FilterSupportedCodecs

This CL removes WebRtcVideoReceiveStream::FilterSupportedCodecs and
adds more fine grained code for selecting and verifying codecs. This
also removes unnecessary copying of cricket::VideoCodecs.

BUG=webrtc:6337
Committed: https://crrev.com/23b7a4a3906dbe9edc3b2188198a1a2c24be3154
Cr-Commit-Position: refs/heads/master@{#14964}

[magjed_webrtc, 2016-11-04 14:09:28]

Patchset #1 (id:1) has been deleted

[magjed_webrtc, 2016-11-04 14:18:37]

Description was changed from

==========
Simplify logic for selecting video send codec

BUG=webrtc:6337
==========

to

==========
Simplify logic for selecting video send codec

This CL simplifies the logic for selecting video send codec and adds
more fine grained control how and when we match video codecs. The
purpose is to prepare for adding High H264 profile support. In an
interim period, we will have to handle older WebRTC versions that do not
follow the spec and will reply with the High H264 profile payload type,
even if they can't handle it. We can detect these cases and send H264
constrained baseline in return. Eventually, we should start rejecting
these replies.

BUG=webrtc:6337
==========

[magjed_webrtc, 2016-11-07 14:24:48]

Patchset #1 (id:20001) has been deleted

[magjed_webrtc, 2016-11-07 14:29:45]

Description was changed from

==========
Simplify logic for selecting video send codec

This CL simplifies the logic for selecting video send codec and adds
more fine grained control how and when we match video codecs. The
purpose is to prepare for adding High H264 profile support. In an
interim period, we will have to handle older WebRTC versions that do not
follow the spec and will reply with the High H264 profile payload type,
even if they can't handle it. We can detect these cases and send H264
constrained baseline in return. Eventually, we should start rejecting
these replies.

BUG=webrtc:6337
==========

to

==========
Refactor WebRtcVideoReceiveStream::FilterSupportedCodecs

This CL removes WebRtcVideoReceiveStream::FilterSupportedCodecs and adds more
fine grained code for selecting and verifying codecs. This also removes
unnecessary copying of cricket::VideoCodecs.

BUG=webrtc:6337
==========

[magjed_webrtc, 2016-11-07 14:29:45]

magjed@webrtc.org changed reviewers:
+ kthelgason@webrtc.org

[magjed_webrtc, 2016-11-07 14:30:43]

Description was changed from

==========
Refactor WebRtcVideoReceiveStream::FilterSupportedCodecs

This CL removes WebRtcVideoReceiveStream::FilterSupportedCodecs and adds more
fine grained code for selecting and verifying codecs. This also removes
unnecessary copying of cricket::VideoCodecs.

BUG=webrtc:6337
==========

to

==========
Refactor WebRtcVideoReceiveStream::FilterSupportedCodecs

This CL removes WebRtcVideoReceiveStream::FilterSupportedCodecs and
adds more fine grained code for selecting and verifying codecs. This
also removes unnecessary copying of cricket::VideoCodecs.

BUG=webrtc:6337
==========

[magjed_webrtc, 2016-11-07 14:30:58]

Kàri - please take a look.

[kthelgason, 2016-11-07 16:28:22]

lgtm

[magjed_webrtc, 2016-11-08 09:00:34]

The CQ bit was checked by magjed@webrtc.org

[commit-bot: I haz the power, 2016-11-08 09:00:41]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2016-11-08 09:12:56]

Description was changed from

==========
Refactor WebRtcVideoReceiveStream::FilterSupportedCodecs

This CL removes WebRtcVideoReceiveStream::FilterSupportedCodecs and
adds more fine grained code for selecting and verifying codecs. This
also removes unnecessary copying of cricket::VideoCodecs.

BUG=webrtc:6337
==========

to

==========
Refactor WebRtcVideoReceiveStream::FilterSupportedCodecs

This CL removes WebRtcVideoReceiveStream::FilterSupportedCodecs and
adds more fine grained code for selecting and verifying codecs. This
also removes unnecessary copying of cricket::VideoCodecs.

BUG=webrtc:6337
==========

[commit-bot: I haz the power, 2016-11-08 09:12:57]

Committed patchset #1 (id:40001)

[commit-bot: I haz the power, 2016-11-08 09:13:06]

Description was changed from

==========
Refactor WebRtcVideoReceiveStream::FilterSupportedCodecs

This CL removes WebRtcVideoReceiveStream::FilterSupportedCodecs and
adds more fine grained code for selecting and verifying codecs. This
also removes unnecessary copying of cricket::VideoCodecs.

BUG=webrtc:6337
==========

to

==========
Refactor WebRtcVideoReceiveStream::FilterSupportedCodecs

This CL removes WebRtcVideoReceiveStream::FilterSupportedCodecs and
adds more fine grained code for selecting and verifying codecs. This
also removes unnecessary copying of cricket::VideoCodecs.

BUG=webrtc:6337

Committed: https://crrev.com/23b7a4a3906dbe9edc3b2188198a1a2c24be3154
Cr-Commit-Position: refs/heads/master@{#14964}
==========

[commit-bot: I haz the power, 2016-11-08 09:13:07]

Patchset 1 (id:??) landed as
https://crrev.com/23b7a4a3906dbe9edc3b2188198a1a2c24be3154
Cr-Commit-Position: refs/heads/master@{#14964}