Android MediaCodecVideoDecoder: Limit measured decode time to 200ms

Android MediaCodecVideoDecoder: Limit measured decode time to 200ms

This change is done to remove abnormally high decode time measurements for H264 decoding. H264 decoding sometimes keeps a few frames as reference before outputting a new decoded frame. This pipeline causes some frames to get stuck when the source stops sending new frames. When the source starts sending frames again, the decode time measurements for the frames that were stuck will include the pause time, which can be arbitrary high. This CL is a simple fix for this problem by constraining the decode time values to a "reasonable" range.

BUG=b/27306053

Committed: https://crrev.com/cedddbdf7b54541ce1786495ab9cc582b3ed58c4
Cr-Commit-Position: refs/heads/master@{#11792}

[magjed_webrtc, 2016-02-26 14:15:53]

Description was changed from

==========
Android MediaCodecVideoDecoder: Limit measured decode time to 200ms

This change is done to remove abnormally high decode time measurements for H264
decoding. H264 decoding sometimes keeps a few frames as reference before
outputting a new decoded frame. This pipeline causes some frames to get stuck
when the source stops sending new frames. When the source starts sending frames
again, the decode time measurements for the frames that were stuck will include
the pause time, which can be arbitrary high. This CL is a simple fix for this
problem by constraining the decode time values to a "reasonable" range.

BUG=b/27306053
==========

to

==========
Android MediaCodecVideoDecoder: Limit measured decode time to 200ms

This change is done to remove abnormally high decode time measurements for H264
decoding. H264 decoding sometimes keeps a few frames as reference before
outputting a new decoded frame. This pipeline causes some frames to get stuck
when the source stops sending new frames. When the source starts sending frames
again, the decode time measurements for the frames that were stuck will include
the pause time, which can be arbitrary high. This CL is a simple fix for this
problem by constraining the decode time values to a "reasonable" range.

BUG=b/27306053
==========

[magjed_webrtc, 2016-02-26 14:15:53]

magjed@webrtc.org changed reviewers:
+ perkj@webrtc.org, stefan@webrtc.org

[magjed_webrtc, 2016-02-26 14:16:36]

Please take a look.

[perkj_webrtc, 2016-02-26 14:28:23]

lgtm

[stefan-webrtc, 2016-02-26 14:36:59]

lgtm

[magjed_webrtc, 2016-02-26 15:52:52]

The CQ bit was checked by magjed@webrtc.org

[commit-bot: I haz the power, 2016-02-26 15:53:01]

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

[commit-bot: I haz the power, 2016-02-26 17:36:07]

Description was changed from

==========
Android MediaCodecVideoDecoder: Limit measured decode time to 200ms

This change is done to remove abnormally high decode time measurements for H264
decoding. H264 decoding sometimes keeps a few frames as reference before
outputting a new decoded frame. This pipeline causes some frames to get stuck
when the source stops sending new frames. When the source starts sending frames
again, the decode time measurements for the frames that were stuck will include
the pause time, which can be arbitrary high. This CL is a simple fix for this
problem by constraining the decode time values to a "reasonable" range.

BUG=b/27306053
==========

to

==========
Android MediaCodecVideoDecoder: Limit measured decode time to 200ms

This change is done to remove abnormally high decode time measurements for H264
decoding. H264 decoding sometimes keeps a few frames as reference before
outputting a new decoded frame. This pipeline causes some frames to get stuck
when the source stops sending new frames. When the source starts sending frames
again, the decode time measurements for the frames that were stuck will include
the pause time, which can be arbitrary high. This CL is a simple fix for this
problem by constraining the decode time values to a "reasonable" range.

BUG=b/27306053
==========

[commit-bot: I haz the power, 2016-02-26 17:36:08]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-26 17:36:18]

Description was changed from

==========
Android MediaCodecVideoDecoder: Limit measured decode time to 200ms

This change is done to remove abnormally high decode time measurements for H264
decoding. H264 decoding sometimes keeps a few frames as reference before
outputting a new decoded frame. This pipeline causes some frames to get stuck
when the source stops sending new frames. When the source starts sending frames
again, the decode time measurements for the frames that were stuck will include
the pause time, which can be arbitrary high. This CL is a simple fix for this
problem by constraining the decode time values to a "reasonable" range.

BUG=b/27306053
==========

to

==========
Android MediaCodecVideoDecoder: Limit measured decode time to 200ms

This change is done to remove abnormally high decode time measurements for H264
decoding. H264 decoding sometimes keeps a few frames as reference before
outputting a new decoded frame. This pipeline causes some frames to get stuck
when the source stops sending new frames. When the source starts sending frames
again, the decode time measurements for the frames that were stuck will include
the pause time, which can be arbitrary high. This CL is a simple fix for this
problem by constraining the decode time values to a "reasonable" range.

BUG=b/27306053

Committed: https://crrev.com/cedddbdf7b54541ce1786495ab9cc582b3ed58c4
Cr-Commit-Position: refs/heads/master@{#11792}
==========

[commit-bot: I haz the power, 2016-02-26 17:36:18]

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