Don't update the jitter estimate with frames containing retransmitted packets.

Don't update the jitter estimate with frames containing retransmitted packets.

BUG=chromium:682636
Review-Url: https://codereview.webrtc.org/2645343002
Cr-Commit-Position: refs/heads/master@{#16273}
Committed: https://chromium.googlesource.com/external/webrtc/+/e0754305aa8787c483c2a4fd319fa383b9bf1493

[philipel, 2017-01-23 11:22:34]

philipel@webrtc.org changed reviewers:
+ stefan@webrtc.org

[stefan-webrtc, 2017-01-24 13:03:14]

https://codereview.webrtc.org/2645343002/diff/60001/webrtc/modules/video_codi...
File webrtc/modules/video_coding/frame_object.h (right):

https://codereview.webrtc.org/2645343002/diff/60001/webrtc/modules/video_codi...
webrtc/modules/video_coding/frame_object.h:42: // TODO(philipel): Remove this
function when new a new timing class has
when a new

https://codereview.webrtc.org/2645343002/diff/60001/webrtc/modules/video_codi...
webrtc/modules/video_coding/frame_object.h:44: virtual int times_nacked() const
{ return 0; }
Should we call it "delayed_by_retransmission()" instead and make it a bool?

[philipel, 2017-01-25 15:04:17]

https://codereview.webrtc.org/2645343002/diff/60001/webrtc/modules/video_codi...
File webrtc/modules/video_coding/frame_object.h (right):

https://codereview.webrtc.org/2645343002/diff/60001/webrtc/modules/video_codi...
webrtc/modules/video_coding/frame_object.h:42: // TODO(philipel): Remove this
function when new a new timing class has
On 2017/01/24 13:03:14, stefan-webrtc wrote:
> when a new

Done.

https://codereview.webrtc.org/2645343002/diff/60001/webrtc/modules/video_codi...
webrtc/modules/video_coding/frame_object.h:44: virtual int times_nacked() const
{ return 0; }
On 2017/01/24 13:03:14, stefan-webrtc wrote:
> Should we call it "delayed_by_retransmission()" instead and make it a bool?

Done.

[stefan-webrtc, 2017-01-25 15:05:27]

lgtm

[philipel, 2017-01-25 15:31:15]

The CQ bit was checked by philipel@webrtc.org

[commit-bot: I haz the power, 2017-01-25 15:31:27]

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-01-25 16:56:23]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1485358275932820,
"parent_rev": "090c9405ccb0bb889e0df8dbc41e8b0446db6325", "commit_rev":
"e0754305aa8787c483c2a4fd319fa383b9bf1493"}

[commit-bot: I haz the power, 2017-01-25 16:56:26]

Description was changed from

==========
Don't update the jitter estimate with frames containing retransmitted packets.

BUG=chromium:682636
==========

to

==========
Don't update the jitter estimate with frames containing retransmitted packets.

BUG=chromium:682636

Review-Url: https://codereview.webrtc.org/2645343002
Cr-Commit-Position: refs/heads/master@{#16273}
Committed:
https://chromium.googlesource.com/external/webrtc/+/e0754305aa8787c483c2a4fd3...
==========

[commit-bot: I haz the power, 2017-01-25 16:56:27]

Committed patchset #5 (id:80001) as
https://chromium.googlesource.com/external/webrtc/+/e0754305aa8787c483c2a4fd3...