Delete Rtx-related methods from RTPPayloadRegistry.

Delete Rtx-related methods from RTPPayloadRegistry.

Delete methods IsRtx, IsEncapsulated and RestoreOriginalPacket.

BUG=webrtc:7135
Review-Url: https://codereview.webrtc.org/3006993002
Cr-Commit-Position: refs/heads/master@{#19739}
Committed: https://chromium.googlesource.com/external/webrtc/+/5b4b52264132eefba10bc6880871715f9692da90

[nisse-webrtc, 2017-09-07 13:09:10]

nisse@webrtc.org changed reviewers:
+ danilchap@webrtc.org, stefan@webrtc.org

[nisse-webrtc, 2017-09-07 13:09:11]

This cl deletes the old code for doing RTX decapsulation.

It's still unclear to me what's the intended "code under test" for
RtpRtcpRtxNackTest. I took out RtpReceiver from the tested code path.

[stefan-webrtc, 2017-09-07 13:22:08]

lgtm

[danilchap, 2017-09-07 13:38:01]

lgtm if all bots agree these functions are no longer used.

"code under test" for RtpRtcpRtxNackTest seems to be RtpRtcp functionality to
reply with rtx packets when nack packets received.
So removing RtpReceiver out off it is a good thing, makes test setup more
straightforward.

[nisse-webrtc, 2017-09-08 08:34:45]

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

[commit-bot: I haz the power, 2017-09-08 08:34:49]

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-09-08 09:07:50]

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

[commit-bot: I haz the power, 2017-09-08 09:07:51]

Dry run: This issue passed the CQ dry run.

[nisse-webrtc, 2017-09-08 09:08:50]

The CQ bit was checked by nisse@webrtc.org

[commit-bot: I haz the power, 2017-09-08 09:09:10]

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-09-08 09:11:55]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1504861730278040,
"parent_rev": "25586cefddcc4b38a5260ef1c6b8cc72b9c1893e", "commit_rev":
"5b4b52264132eefba10bc6880871715f9692da90"}

[commit-bot: I haz the power, 2017-09-08 09:12:00]

Description was changed from

==========
Delete Rtx-related methods from RTPPayloadRegistry.

Delete methods IsRtx, IsEncapsulated and RestoreOriginalPacket.

BUG=webrtc:7135
==========

to

==========
Delete Rtx-related methods from RTPPayloadRegistry.

Delete methods IsRtx, IsEncapsulated and RestoreOriginalPacket.

BUG=webrtc:7135

Review-Url: https://codereview.webrtc.org/3006993002
Cr-Commit-Position: refs/heads/master@{#19739}
Committed:
https://chromium.googlesource.com/external/webrtc/+/5b4b52264132eefba10bc6880...
==========

[commit-bot: I haz the power, 2017-09-08 09:12:02]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/external/webrtc/+/5b4b52264132eefba10bc6880...

[nisse-webrtc, 2017-09-08 12:00:37]

A revert of this CL (patchset #3 id:40001) has been created in
https://codereview.webrtc.org/3011093002/ by nisse@webrtc.org.

The reason for reverting is: This has to be reverted to enable reverting cl
https://codereview.webrtc.org/3006063002/, which seems to have broken ulpfec..