Implement PlayoutDelay extension as a trait

Implement PlayoutDelay extension as a trait
to be used with rtp::Packet class

BUG=webrtc:1994
R=isheriff@chromium.org, stefan@webrtc.org

Committed: https://crrev.com/08b0351ddd0469c8b0aa7c028ac93878a350e593
Cr-Commit-Position: refs/heads/master@{#14105}

[danilchap, 2016-08-09 11:20:35]

Description was changed from

==========
Implement PlayoutDelay extension as a trait
to be used with rtp::Packet class

BUG=webrtc:1994
==========

to

==========
Implement PlayoutDelay extension as a trait
to be used with rtp::Packet class

BUG=webrtc:1994
==========

[danilchap, 2016-08-15 14:18:30]

danilchap@webrtc.org changed reviewers:
+ isheriff@chromium.org, stefan@webrtc.org

[danilchap, 2016-08-15 14:18:31]

Before spreading RtpPacket class to RtpSenderVideo, need it to support
PlayoutDelay rtp header extension in provided form.

[Irfan, 2016-08-16 17:22:14]

https://codereview.chromium.org/2224063004/diff/40001/webrtc/modules/rtp_rtcp...
File webrtc/modules/rtp_rtcp/source/rtp_header_extensions.h (right):

https://codereview.chromium.org/2224063004/diff/40001/webrtc/modules/rtp_rtcp...
webrtc/modules/rtp_rtcp/source/rtp_header_extensions.h:87: static bool
Parse(const uint8_t* data, int* min_ms, int* max_ms);
Parse it into PlayoutDelay variable ?

https://codereview.chromium.org/2224063004/diff/40001/webrtc/modules/rtp_rtcp...
webrtc/modules/rtp_rtcp/source/rtp_header_extensions.h:88: static bool
Write(uint8_t* data, int min_ms, int max_ms);
Similarly, Write a PlayoutDelay variable ?

[danilchap, 2016-08-17 07:42:39]

https://codereview.webrtc.org/2224063004/diff/40001/webrtc/modules/rtp_rtcp/s...
File webrtc/modules/rtp_rtcp/source/rtp_header_extensions.h (right):

https://codereview.webrtc.org/2224063004/diff/40001/webrtc/modules/rtp_rtcp/s...
webrtc/modules/rtp_rtcp/source/rtp_header_extensions.h:87: static bool
Parse(const uint8_t* data, int* min_ms, int* max_ms);
On 2016/08/16 17:22:14, Irfan wrote:
> Parse it into PlayoutDelay variable ?

Done.

https://codereview.webrtc.org/2224063004/diff/40001/webrtc/modules/rtp_rtcp/s...
webrtc/modules/rtp_rtcp/source/rtp_header_extensions.h:88: static bool
Write(uint8_t* data, int min_ms, int max_ms);
On 2016/08/16 17:22:13, Irfan wrote:
> Similarly, Write a PlayoutDelay variable ?

Done.

[Irfan, 2016-08-17 16:41:38]

lgtm

[danilchap, 2016-08-22 06:37:32]

ping Stefan
Can I please get an approve for webrtc/test/ ?

[stefan-webrtc, 2016-09-07 13:02:52]

lgtm

[danilchap, 2016-09-07 13:08:23]

Description was changed from

==========
Implement PlayoutDelay extension as a trait
to be used with rtp::Packet class

BUG=webrtc:1994
==========

to

==========
Implement PlayoutDelay extension as a trait
to be used with rtp::Packet class

BUG=webrtc:1994
R=isheriff@chromium.org, stefan@webrtc.org

Committed:
https://chromium.googlesource.com/external/webrtc/+/08b0351ddd0469c8b0aa7c028...
==========

[danilchap, 2016-09-07 13:08:25]

Committed patchset #4 (id:60001) manually as
08b0351ddd0469c8b0aa7c028ac93878a350e593 (presubmit successful).

[commit-bot: I haz the power, 2016-09-07 13:08:27]

Description was changed from

==========
Implement PlayoutDelay extension as a trait
to be used with rtp::Packet class

BUG=webrtc:1994
R=isheriff@chromium.org, stefan@webrtc.org

Committed:
https://chromium.googlesource.com/external/webrtc/+/08b0351ddd0469c8b0aa7c028...
==========

to

==========
Implement PlayoutDelay extension as a trait
to be used with rtp::Packet class

BUG=webrtc:1994
R=isheriff@chromium.org, stefan@webrtc.org

Committed: https://crrev.com/08b0351ddd0469c8b0aa7c028ac93878a350e593
Cr-Commit-Position: refs/heads/master@{#14105}
==========