[rtp_rtcp] rtcp::Empty moved into own file and renamed to CompoundPacket on the way

[rtp_rtcp] rtcp::Empty moved into own file and renamed to CompoundPacket on the way

Class renamed to indicated use of the rtcp::Empty class: it can only be used as container for other rtcp packets.
All tests that use Append function moved from rtcp_packet_unittest, even if they did not use Empty class.
This is because there is plan to make RtcpPacket class lighter by moving Append functionality to CompoundPacket class.

BUG=webrtc:5260
R=åsapersson

Committed: https://crrev.com/2f7dea164dc49ae8a0322e3c9edb1dd23266c664
Cr-Commit-Position: refs/heads/master@{#11234}

[åsapersson, 2016-01-12 13:07:19]

lgtm

https://codereview.webrtc.org/1582503002/diff/1/webrtc/modules/rtp_rtcp/sourc...
File webrtc/modules/rtp_rtcp/source/rtcp_packet/compound_packet_unittest.cc
(right):

https://codereview.webrtc.org/1582503002/diff/1/webrtc/modules/rtp_rtcp/sourc...
webrtc/modules/rtp_rtcp/source/rtcp_packet/compound_packet_unittest.cc:10: *
This file includes unit tests for the RtcpPacket.
perhaps remove this comment

https://codereview.webrtc.org/1582503002/diff/1/webrtc/modules/rtp_rtcp/sourc...
webrtc/modules/rtp_rtcp/source/rtcp_packet/compound_packet_unittest.cc:34:
TEST(RtcpPacketTest, AppendPacket) {
rename to RtcpPacketCompoundTest?

[danilchap, 2016-01-12 13:20:49]

https://codereview.webrtc.org/1582503002/diff/1/webrtc/modules/rtp_rtcp/sourc...
File webrtc/modules/rtp_rtcp/source/rtcp_packet/compound_packet_unittest.cc
(right):

https://codereview.webrtc.org/1582503002/diff/1/webrtc/modules/rtp_rtcp/sourc...
webrtc/modules/rtp_rtcp/source/rtcp_packet/compound_packet_unittest.cc:10: *
This file includes unit tests for the RtcpPacket.
On 2016/01/12 13:07:19, åsapersson wrote:
> perhaps remove this comment

Done.

https://codereview.webrtc.org/1582503002/diff/1/webrtc/modules/rtp_rtcp/sourc...
webrtc/modules/rtp_rtcp/source/rtcp_packet/compound_packet_unittest.cc:34:
TEST(RtcpPacketTest, AppendPacket) {
On 2016/01/12 13:07:19, åsapersson wrote:
> rename to RtcpPacketCompoundTest?

Done.

[danilchap, 2016-01-13 09:35:26]

The CQ bit was checked by danilchap@webrtc.org

[danilchap, 2016-01-13 09:35:27]

The patchset sent to the CQ was uploaded after l-g-t-m from
asapersson@webrtc.org
Link to the patchset: https://codereview.webrtc.org/1582503002/#ps20001 (title:
" ")

[commit-bot: I haz the power, 2016-01-13 09:35:37]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1582503002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1582503002/20001

[commit-bot: I haz the power, 2016-01-13 10:03:08]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-01-13 10:03:16]

Description was changed from

==========
[rtp_rtcp] rtcp::Empty moved into own file and renamed to CompoundPacket on the
way

Class renamed to indicated use of the rtcp::Empty class: it can only be used as
container for other rtcp packets.
All tests that use Append function moved from rtcp_packet_unittest, even if they
did not use Empty class.
This is because there is plan to make RtcpPacket class lighter by moving Append
functionality to CompoundPacket class.

BUG=webrtc:5260
R=åsapersson
==========

to

==========
[rtp_rtcp] rtcp::Empty moved into own file and renamed to CompoundPacket on the
way

Class renamed to indicated use of the rtcp::Empty class: it can only be used as
container for other rtcp packets.
All tests that use Append function moved from rtcp_packet_unittest, even if they
did not use Empty class.
This is because there is plan to make RtcpPacket class lighter by moving Append
functionality to CompoundPacket class.

BUG=webrtc:5260
R=åsapersson

Committed: https://crrev.com/2f7dea164dc49ae8a0322e3c9edb1dd23266c664
Cr-Commit-Position: refs/heads/master@{#11234}
==========

[commit-bot: I haz the power, 2016-01-13 10:03:17]

Patchset 2 (id:??) landed as
https://crrev.com/2f7dea164dc49ae8a0322e3c9edb1dd23266c664
Cr-Commit-Position: refs/heads/master@{#11234}