Remove cricket::VideoCodec with, height and framerate properties

Remove cricket::VideoCodec with, height and framerate properties
Since WebRtcVideoSendStream have reconfigures the send codec to match the incoming captured frames widht and height they have not been used.
Framerate has just been set when parsing sdp to 60fps and not changed elsewhere.

This cl require some upstream projects to change first.

BUG=webrtc:5332
Committed: https://crrev.com/267527459bd929a770278441493d4ec3da81de03
Cr-Commit-Position: refs/heads/master@{#14733}

[perkj_webrtc, 2016-10-11 13:56:04]

Description was changed from

==========
Removed VideoCodec with, height and framerate properties
Since WebRtcVideoSendStream have reconfigures the send codec to match the
incoming captured frames widht and height they have not been used.
Framerate has just been set when parsing sdp to 60fps and not changed elsewhere.

This cl require some upstream projects to change first.

BUG=webrtc:5332
==========

to

==========
Remove cricket::VideoCodec with, height and framerate properties
Since WebRtcVideoSendStream have reconfigures the send codec to match the
incoming captured frames widht and height they have not been used.
Framerate has just been set when parsing sdp to 60fps and not changed elsewhere.

This cl require some upstream projects to change first.

BUG=webrtc:5332
==========

[perkj_webrtc, 2016-10-11 17:29:34]

perkj@webrtc.org changed reviewers:
+ pthatcher@webrtc.org

[pthatcher1, 2016-10-13 21:33:47]

lgtm

I love seeing this finally happen.

https://codereview.webrtc.org/2408153002/diff/40001/webrtc/api/jsepsessiondes...
File webrtc/api/jsepsessiondescription.h (right):

https://codereview.webrtc.org/2408153002/diff/40001/webrtc/api/jsepsessiondes...
webrtc/api/jsepsessiondescription.h:70: static const int
kDefaultVideoCodecFramerate;
Shouldn't this be removed also?

[perkj_webrtc, 2016-10-18 12:09:40]

https://codereview.webrtc.org/2408153002/diff/40001/webrtc/api/jsepsessiondes...
File webrtc/api/jsepsessiondescription.h (right):

https://codereview.webrtc.org/2408153002/diff/40001/webrtc/api/jsepsessiondes...
webrtc/api/jsepsessiondescription.h:70: static const int
kDefaultVideoCodecFramerate;
On 2016/10/13 21:33:47, pthatcher1 wrote:
> Shouldn't this be removed also?

yes, thanks.

[perkj_webrtc, 2016-10-24 07:13:18]

The CQ bit was checked by perkj@webrtc.org

[perkj_webrtc, 2016-10-24 07:13:18]

The patchset sent to the CQ was uploaded after l-g-t-m from pthatcher@webrtc.org
Link to the patchset: https://codereview.chromium.org/2408153002/#ps80001
(title: "Rebased")

[commit-bot: I haz the power, 2016-10-24 07:13:27]

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-10-24 08:21:20]

Description was changed from

==========
Remove cricket::VideoCodec with, height and framerate properties
Since WebRtcVideoSendStream have reconfigures the send codec to match the
incoming captured frames widht and height they have not been used.
Framerate has just been set when parsing sdp to 60fps and not changed elsewhere.

This cl require some upstream projects to change first.

BUG=webrtc:5332
==========

to

==========
Remove cricket::VideoCodec with, height and framerate properties
Since WebRtcVideoSendStream have reconfigures the send codec to match the
incoming captured frames widht and height they have not been used.
Framerate has just been set when parsing sdp to 60fps and not changed elsewhere.

This cl require some upstream projects to change first.

BUG=webrtc:5332
==========

[commit-bot: I haz the power, 2016-10-24 08:21:21]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-10-24 08:21:27]

Description was changed from

==========
Remove cricket::VideoCodec with, height and framerate properties
Since WebRtcVideoSendStream have reconfigures the send codec to match the
incoming captured frames widht and height they have not been used.
Framerate has just been set when parsing sdp to 60fps and not changed elsewhere.

This cl require some upstream projects to change first.

BUG=webrtc:5332
==========

to

==========
Remove cricket::VideoCodec with, height and framerate properties
Since WebRtcVideoSendStream have reconfigures the send codec to match the
incoming captured frames widht and height they have not been used.
Framerate has just been set when parsing sdp to 60fps and not changed elsewhere.

This cl require some upstream projects to change first.

BUG=webrtc:5332

Committed: https://crrev.com/267527459bd929a770278441493d4ec3da81de03
Cr-Commit-Position: refs/heads/master@{#14733}
==========

[commit-bot: I haz the power, 2016-10-24 08:21:28]

Patchset 5 (id:??) landed as
https://crrev.com/267527459bd929a770278441493d4ec3da81de03
Cr-Commit-Position: refs/heads/master@{#14733}