Refactor RTX video codec and payload type assignment

Refactor RTX video codec and payload type assignment

We want to reuse some of this functionality for the new video codec
factories, but not of all it, so this CL refactors out what we want to
reuse to a static function.

BUG=webrtc:7925
Review-Url: https://codereview.webrtc.org/3010743002
Cr-Commit-Position: refs/heads/master@{#19628}
Committed: https://chromium.googlesource.com/external/webrtc/+/6ed63255cab783e7dbeea8a8deadfc27fa6abde7

[magjed_webrtc, 2017-08-30 12:56:01]

The CQ bit was checked by magjed@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-08-30 12:56:10]

Dry run: 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, 2017-08-30 13:47:22]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-08-30 13:47:23]

Dry run: This issue passed the CQ dry run.

[magjed_webrtc, 2017-08-30 14:07:38]

The CQ bit was checked by magjed@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-08-30 14:07:49]

Dry run: CQ is trying da patch.

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

[magjed_webrtc, 2017-08-30 14:33:43]

Patchset #1 (id:1) has been deleted

[magjed_webrtc, 2017-08-30 14:38:08]

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

[magjed_webrtc, 2017-08-30 14:38:09]

Anders - please take a look.

[commit-bot: I haz the power, 2017-08-30 14:53:01]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-08-30 14:53:01]

Dry run: This issue passed the CQ dry run.

[andersc, 2017-08-31 08:52:21]

On 2017/08/30 14:38:09, magjed_webrtc wrote:
> Anders - please take a look.

Nice, I find the new code much easier to follow. lgtm

[magjed_webrtc, 2017-08-31 12:34:22]

The CQ bit was checked by magjed@webrtc.org

[commit-bot: I haz the power, 2017-08-31 12:34:34]

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, 2017-08-31 12:37:06]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1504182862539550,
"parent_rev": "5fcd1cc4bbba639b1544c3ae9640ac45bfd1ff44", "commit_rev":
"6ed63255cab783e7dbeea8a8deadfc27fa6abde7"}

[commit-bot: I haz the power, 2017-08-31 12:37:10]

Description was changed from

==========
Refactor RTX video codec and payload type assignment

We want to reuse some of this functionality for the new video codec
factories, but not of all it, so this CL refactors out what we want to
reuse to a static function.

BUG=webrtc:7925
==========

to

==========
Refactor RTX video codec and payload type assignment

We want to reuse some of this functionality for the new video codec
factories, but not of all it, so this CL refactors out what we want to
reuse to a static function.

BUG=webrtc:7925

Review-Url: https://codereview.webrtc.org/3010743002
Cr-Commit-Position: refs/heads/master@{#19628}
Committed:
https://chromium.googlesource.com/external/webrtc/+/6ed63255cab783e7dbeea8a8d...
==========

[commit-bot: I haz the power, 2017-08-31 12:37:11]

Committed patchset #1 (id:20001) as
https://chromium.googlesource.com/external/webrtc/+/6ed63255cab783e7dbeea8a8d...