VTVDA: Fix computation of |reorder_window|.

VTVDA: Fix computation of |reorder_window|.

Prior to this change, VTVDA would disable reordering when there was no
VUI max_num_reorder_frames syntax element. This is incorrect; excluding
an explicit list of profiles, it should be MaxDpbFrames.

The incorrect behavior is masked during normal playback, because
decoding is typically faster than rendering. This causes the reorder
queue to fill up even though reordering is supposed to be turned off,
and frames therefore get a chance to be ordered correctly.

BUG=634007
Committed: https://crrev.com/8cbe5d69e18a9c387cfc61adf826ae121cf89834
Cr-Commit-Position: refs/heads/master@{#414499}

[sandersd (OOO until July 31), 2016-08-25 00:39:40]

The CQ bit was checked by sandersd@chromium.org to run a CQ dry run

[sandersd (OOO until July 31), 2016-08-25 00:40:16]

sandersd@chromium.org changed reviewers:
+ posciak@chromium.org

[sandersd (OOO until July 31), 2016-08-25 00:40:17]

posciak@: If everything looks good, please CQ.

[commit-bot: I haz the power, 2016-08-25 00:41:03]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[sandersd (OOO until July 31), 2016-08-25 00:41:39]

Description was changed from

==========
VTVDA: Fix computation of |reorder_window|.

Prior to this change, VTVDA would disable reordering when there was no
VUI |max_num_reorder_frames| syntax element. This is incorrect; except
for an explicit list of profiles it should be MaxDpbFrames.

The incorrect behavior is masked during normal playback, because
decoding is typically faster than rendering. This causes the reorder
queue to fill up even though reordering was supposed to be turned off,
and frames therefore got a chance to be ordered correctly.

BUG=634007
==========

to

==========
VTVDA: Fix computation of |reorder_window|.

Prior to this change, VTVDA would disable reordering when there was no
VUI max_num_reorder_frames syntax element. This is incorrect; excluding
an explicit list of profiles, it should be MaxDpbFrames.

The incorrect behavior is masked during normal playback, because
decoding is typically faster than rendering. This causes the reorder
queue to fill up even though reordering is supposed to be turned off,
and frames therefore get a chance to be ordered correctly.

BUG=634007
==========

[commit-bot: I haz the power, 2016-08-25 02:43:30]

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

[commit-bot: I haz the power, 2016-08-25 02:43:31]

Dry run: This issue passed the CQ dry run.

[sandersd (OOO until July 31), 2016-08-25 17:05:42]

sandersd@chromium.org changed reviewers:
+ dalecurtis@chromium.org

[sandersd (OOO until July 31), 2016-08-25 17:05:43]

sandersd@chromium.org changed required reviewers:
+ dalecurtis@chromium.org

[sandersd (OOO until July 31), 2016-08-25 17:06:00]

-> dalecurtis@ since I'm trying to commit this before the branch.

[DaleCurtis, 2016-08-25 17:55:34]

lgtm, seems like this needs a test though.

[sandersd (OOO until July 31), 2016-08-25 18:57:37]

On 2016/08/25 17:55:34, DaleCurtis wrote:
> lgtm, seems like this needs a test though.

As discussed offline, the plan is to extract this (either into h264_poc or a
sibling) as a followup.

[sandersd (OOO until July 31), 2016-08-25 18:57:44]

The CQ bit was checked by sandersd@chromium.org

[commit-bot: I haz the power, 2016-08-25 18:58:38]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-25 19:04:42]

Description was changed from

==========
VTVDA: Fix computation of |reorder_window|.

Prior to this change, VTVDA would disable reordering when there was no
VUI max_num_reorder_frames syntax element. This is incorrect; excluding
an explicit list of profiles, it should be MaxDpbFrames.

The incorrect behavior is masked during normal playback, because
decoding is typically faster than rendering. This causes the reorder
queue to fill up even though reordering is supposed to be turned off,
and frames therefore get a chance to be ordered correctly.

BUG=634007
==========

to

==========
VTVDA: Fix computation of |reorder_window|.

Prior to this change, VTVDA would disable reordering when there was no
VUI max_num_reorder_frames syntax element. This is incorrect; excluding
an explicit list of profiles, it should be MaxDpbFrames.

The incorrect behavior is masked during normal playback, because
decoding is typically faster than rendering. This causes the reorder
queue to fill up even though reordering is supposed to be turned off,
and frames therefore get a chance to be ordered correctly.

BUG=634007
==========

[commit-bot: I haz the power, 2016-08-25 19:04:43]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-25 19:06:34]

Description was changed from

==========
VTVDA: Fix computation of |reorder_window|.

Prior to this change, VTVDA would disable reordering when there was no
VUI max_num_reorder_frames syntax element. This is incorrect; excluding
an explicit list of profiles, it should be MaxDpbFrames.

The incorrect behavior is masked during normal playback, because
decoding is typically faster than rendering. This causes the reorder
queue to fill up even though reordering is supposed to be turned off,
and frames therefore get a chance to be ordered correctly.

BUG=634007
==========

to

==========
VTVDA: Fix computation of |reorder_window|.

Prior to this change, VTVDA would disable reordering when there was no
VUI max_num_reorder_frames syntax element. This is incorrect; excluding
an explicit list of profiles, it should be MaxDpbFrames.

The incorrect behavior is masked during normal playback, because
decoding is typically faster than rendering. This causes the reorder
queue to fill up even though reordering is supposed to be turned off,
and frames therefore get a chance to be ordered correctly.

BUG=634007

Committed: https://crrev.com/8cbe5d69e18a9c387cfc61adf826ae121cf89834
Cr-Commit-Position: refs/heads/master@{#414499}
==========

[commit-bot: I haz the power, 2016-08-25 19:06:34]

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