Fix a bug in RtpFileSource related to RTCP packets in rtpdump files

Fix a bug in RtpFileSource related to RTCP packets in rtpdump files

According to http://www.cs.columbia.edu/irt/software/rtptools/#rtpdump,
RTCP packets are marked with plen==0. In this class, plen is mapped to
original_length, not length.

Committed: https://crrev.com/11d583f41484913fd1e7b3e283966eb7b7e11ed2
Cr-Commit-Position: refs/heads/master@{#9981}

[hlundin-webrtc, 2015-09-17 09:12:09]

henrik.lundin@webrtc.org changed reviewers:
+ kwiberg@webrtc.org

[hlundin-webrtc, 2015-09-17 09:16:41]

⌛

[kwiberg-webrtc, 2015-09-17 10:51:44]

lgtm

https://codereview.webrtc.org/1356543002/diff/1/webrtc/modules/audio_coding/n...
File webrtc/modules/audio_coding/neteq/tools/rtp_file_source.cc (right):

https://codereview.webrtc.org/1356543002/diff/1/webrtc/modules/audio_coding/n...
webrtc/modules/audio_coding/neteq/tools/rtp_file_source.cc:51: // May be an RTCP
packet.
Tangent: What could it be besides an RTCP packet?

[hlundin-webrtc, 2015-09-17 13:33:07]

https://codereview.webrtc.org/1356543002/diff/1/webrtc/modules/audio_coding/n...
File webrtc/modules/audio_coding/neteq/tools/rtp_file_source.cc (right):

https://codereview.webrtc.org/1356543002/diff/1/webrtc/modules/audio_coding/n...
webrtc/modules/audio_coding/neteq/tools/rtp_file_source.cc:51: // May be an RTCP
packet.
On 2015/09/17 10:51:44, kwiberg-webrtc wrote:
> Tangent: What could it be besides an RTCP packet?

An RTP packet in disguise? :)

[hlundin-webrtc, 2015-09-17 13:33:12]

The CQ bit was checked by henrik.lundin@webrtc.org

[commit-bot: I haz the power, 2015-09-17 13:33:23]

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

[kwiberg-webrtc, 2015-09-17 13:39:32]

https://codereview.webrtc.org/1356543002/diff/1/webrtc/modules/audio_coding/n...
File webrtc/modules/audio_coding/neteq/tools/rtp_file_source.cc (right):

https://codereview.webrtc.org/1356543002/diff/1/webrtc/modules/audio_coding/n...
webrtc/modules/audio_coding/neteq/tools/rtp_file_source.cc:51: // May be an RTCP
packet.
On 2015/09/17 13:33:07, hlundin-webrtc wrote:
> On 2015/09/17 10:51:44, kwiberg-webrtc wrote:
> > Tangent: What could it be besides an RTCP packet?
> 
> An RTP packet in disguise? :)

Or a wolf in an RTP packet's clothing?

[commit-bot: I haz the power, 2015-09-17 15:33:40]

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

[commit-bot: I haz the power, 2015-09-17 15:33:41]

Try jobs failed on following builders:
  win_drmemory_light on tryserver.webrtc (JOB_TIMED_OUT, no build URL)

[hlundin-webrtc, 2015-09-18 08:27:03]

The CQ bit was checked by henrik.lundin@webrtc.org

[commit-bot: I haz the power, 2015-09-18 08:27:13]

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

[commit-bot: I haz the power, 2015-09-18 08:28:10]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-09-18 08:28:25]

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