Revert of Remove RTPPayloadStrategy and simplify RTPPayloadRegistry

Revert of Remove RTPPayloadStrategy and simplify RTPPayloadRegistry (patchset #7 id:240001 of https://codereview.webrtc.org/2524923002/ )

Reason for revert:
Breaks downstream projects.

Original issue's description:
> Remove RTPPayloadStrategy and simplify RTPPayloadRegistry
>
> This CL removes RTPPayloadStrategy that is currently used to handle
> audio/video specific aspects of payload handling. Instead, the audio and
> video specific aspects will now have different functions, with linear
> code flow.
>
> This CL does not contain any functional changes, and is just a
> preparation for future CL:s.
>
> The main purpose with this CL is to add this function:
> bool PayloadIsCompatible(const RtpUtility::Payload& payload,
>                          const webrtc::VideoCodec& video_codec);
> that can easily be extended in a future CL to look at video codec
> specific information.
>
> BUG=webrtc:6743
>
> Committed: https://crrev.com/b881254dc86d2cc80a52e08155433458be002166
> Cr-Commit-Position: refs/heads/master@{#15232}

TBR=danilchap@webrtc.org,solenberg@webrtc.org,mflodman@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:6743
Committed: https://crrev.com/33c81d05613f45f65ee17224ed381c6cdd1c6c6f
Cr-Commit-Position: refs/heads/master@{#15234}

[magjed_webrtc, 2016-11-24 19:08:31]

The CQ bit was checked by magjed@webrtc.org

[magjed_webrtc, 2016-11-24 19:08:31]

Created Revert of Remove RTPPayloadStrategy and simplify RTPPayloadRegistry

[commit-bot: I haz the power, 2016-11-24 19:08:33]

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-24 19:08:40]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1480014511118010, "parent_rev":
"eeb04631fd6ede0291204782683fb86e7d8e8d31", "commit_rev":
"08fe42ea12243388207c2654ac9ca4c03b07a801"}

[commit-bot: I haz the power, 2016-11-24 19:08:44]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-24 19:08:49]

Description was changed from

==========
Revert of Remove RTPPayloadStrategy and simplify RTPPayloadRegistry (patchset #7
id:240001 of https://codereview.webrtc.org/2524923002/ )

Reason for revert:
Breaks downstream projects.

Original issue's description:
> Remove RTPPayloadStrategy and simplify RTPPayloadRegistry
> 
> This CL removes RTPPayloadStrategy that is currently used to handle
> audio/video specific aspects of payload handling. Instead, the audio and
> video specific aspects will now have different functions, with linear
> code flow.
> 
> This CL does not contain any functional changes, and is just a
> preparation for future CL:s.
> 
> The main purpose with this CL is to add this function:
> bool PayloadIsCompatible(const RtpUtility::Payload& payload,
>                          const webrtc::VideoCodec& video_codec);
> that can easily be extended in a future CL to look at video codec
> specific information.
> 
> BUG=webrtc:6743
> 
> Committed: https://crrev.com/b881254dc86d2cc80a52e08155433458be002166
> Cr-Commit-Position: refs/heads/master@{#15232}

TBR=danilchap@webrtc.org,solenberg@webrtc.org,mflodman@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:6743
==========

to

==========
Revert of Remove RTPPayloadStrategy and simplify RTPPayloadRegistry (patchset #7
id:240001 of https://codereview.webrtc.org/2524923002/ )

Reason for revert:
Breaks downstream projects.

Original issue's description:
> Remove RTPPayloadStrategy and simplify RTPPayloadRegistry
>
> This CL removes RTPPayloadStrategy that is currently used to handle
> audio/video specific aspects of payload handling. Instead, the audio and
> video specific aspects will now have different functions, with linear
> code flow.
>
> This CL does not contain any functional changes, and is just a
> preparation for future CL:s.
>
> The main purpose with this CL is to add this function:
> bool PayloadIsCompatible(const RtpUtility::Payload& payload,
>                          const webrtc::VideoCodec& video_codec);
> that can easily be extended in a future CL to look at video codec
> specific information.
>
> BUG=webrtc:6743
>
> Committed: https://crrev.com/b881254dc86d2cc80a52e08155433458be002166
> Cr-Commit-Position: refs/heads/master@{#15232}

TBR=danilchap@webrtc.org,solenberg@webrtc.org,mflodman@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:6743

Committed: https://crrev.com/33c81d05613f45f65ee17224ed381c6cdd1c6c6f
Cr-Commit-Position: refs/heads/master@{#15234}
==========

[commit-bot: I haz the power, 2016-11-24 19:08:50]

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

[magjed_webrtc, 2016-11-25 12:20:36]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.webrtc.org/2531043002/ by magjed@webrtc.org.

The reason for reverting is: Downstream code has been updated..