Defer AVDA codec creation during reset.

Defer AVDA codec creation during reset.

When seeking, AVDA needs to flush or replace the codec instance.
Previously, it would do so on every AVDA::Reset, which would cause
~8 codecs/sec while scrubbing.

This CL causes AVDA to defer codec destruction for Reset() like it
already does for flush.  The result is ~2 codecs / second.

BUG=598385
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2789773002
Cr-Commit-Position: refs/heads/master@{#460941}
Committed: https://chromium.googlesource.com/chromium/src/+/8bf705dd67877f6708eb45ba35ced816013eba5f

[liberato (no reviews please), 2017-03-30 21:51:06]

Description was changed from

==========
Defer AVDA codec creation during reset.

When seeking, AVDA needs to flush or replace the codec instance.
Previously, it would do so on every AVDA::Reset, which would cause
~8 codecs/sec while scrubbing.

This CL causes AVDA to defer codec destruction for Reset() like it
already does for flush.  The result is ~2 codecs / second.

BUG=598385
==========

to

==========
Defer AVDA codec creation during reset.

When seeking, AVDA needs to flush or replace the codec instance.
Previously, it would do so on every AVDA::Reset, which would cause
~8 codecs/sec while scrubbing.

This CL causes AVDA to defer codec destruction for Reset() like it
already does for flush.  The result is ~2 codecs / second.

BUG=598385
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[liberato (no reviews please), 2017-03-30 21:51:29]

Description was changed from

==========
Defer AVDA codec creation during reset.

When seeking, AVDA needs to flush or replace the codec instance.
Previously, it would do so on every AVDA::Reset, which would cause
~8 codecs/sec while scrubbing.

This CL causes AVDA to defer codec destruction for Reset() like it
already does for flush.  The result is ~2 codecs / second.

BUG=598385
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Defer AVDA codec creation during reset.

When seeking, AVDA needs to flush or replace the codec instance.
Previously, it would do so on every AVDA::Reset, which would cause
~8 codecs/sec while scrubbing.

This CL causes AVDA to defer codec destruction for Reset() like it
already does for flush.  The result is ~2 codecs / second.

BUG=598385
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[liberato (no reviews please), 2017-03-30 21:51:29]

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

[liberato (no reviews please), 2017-03-30 21:51:58]

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

[commit-bot: I haz the power, 2017-03-30 21:53:03]

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

[DaleCurtis, 2017-03-30 22:02:49]

lgtm, I thought this had changed since inception, but it hasn't, and I don't
remember why I didn't include it when I wrote the original CL to only do this on
a Flush().

[liberato (no reviews please), 2017-03-30 22:51:51]

The CQ bit was unchecked by liberato@chromium.org

[liberato (no reviews please), 2017-03-30 22:51:54]

The CQ bit was checked by liberato@chromium.org

[commit-bot: I haz the power, 2017-03-30 22:52:22]

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, 2017-03-31 00:01:36]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490914314620830, "parent_rev":
"db2637ff32cb9a619f1e2920293eb2e68f20ac4c", "commit_rev":
"8bf705dd67877f6708eb45ba35ced816013eba5f"}

[commit-bot: I haz the power, 2017-03-31 00:54:39]

Description was changed from

==========
Defer AVDA codec creation during reset.

When seeking, AVDA needs to flush or replace the codec instance.
Previously, it would do so on every AVDA::Reset, which would cause
~8 codecs/sec while scrubbing.

This CL causes AVDA to defer codec destruction for Reset() like it
already does for flush.  The result is ~2 codecs / second.

BUG=598385
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Defer AVDA codec creation during reset.

When seeking, AVDA needs to flush or replace the codec instance.
Previously, it would do so on every AVDA::Reset, which would cause
~8 codecs/sec while scrubbing.

This CL causes AVDA to defer codec destruction for Reset() like it
already does for flush.  The result is ~2 codecs / second.

BUG=598385
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2789773002
Cr-Commit-Position: refs/heads/master@{#460941}
Committed:
https://chromium.googlesource.com/chromium/src/+/8bf705dd67877f6708eb45ba35ce...
==========

[commit-bot: I haz the power, 2017-03-31 00:54:40]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/8bf705dd67877f6708eb45ba35ce...