Disable idle suspend for GpuVideoDecoder produced frames on OSX, Win.

Disable idle suspend for GpuVideoDecoder produced frames on OSX, Win.

The hardware decoders on these platforms can not handle being killed
while there are outstanding frames. Disable for now.

BUG=595716, 602708
.
TEST=suspend does not occur on OSX, Windows.

Committed: https://crrev.com/7f366b224dbd21f8b8e3d93b3a4635bf68f31977
Cr-Commit-Position: refs/heads/master@{#386881}

[DaleCurtis, 2016-04-12 22:37:42]

Patchset #1 (id:1) has been deleted

[DaleCurtis, 2016-04-12 22:37:47]

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

[sandersd (OOO until July 31), 2016-04-12 23:19:19]

lgtm

https://codereview.chromium.org/1879253002/diff/20001/media/blink/webmediapla...
File media/blink/webmediaplayer_impl.cc (right):

https://codereview.chromium.org/1879253002/diff/20001/media/blink/webmediapla...
media/blink/webmediaplayer_impl.cc:1083: #if defined(OS_MACOSX) ||
defined(OS_WIN)
Move this check into SetSuspendState(), now that there is only one place that
Suspend() is called.

[DaleCurtis, 2016-04-12 23:33:29]

https://codereview.chromium.org/1879253002/diff/20001/media/blink/webmediapla...
File media/blink/webmediaplayer_impl.cc (right):

https://codereview.chromium.org/1879253002/diff/20001/media/blink/webmediapla...
media/blink/webmediaplayer_impl.cc:1083: #if defined(OS_MACOSX) ||
defined(OS_WIN)
On 2016/04/12 at 23:19:19, sandersd wrote:
> Move this check into SetSuspendState(), now that there is only one place that
Suspend() is called.

Done. I was surprised to see that SetSuspendState() is called frequently, but
that's fine.

[DaleCurtis, 2016-04-12 23:33:32]

The CQ bit was checked by dalecurtis@chromium.org

[DaleCurtis, 2016-04-12 23:33:32]

The patchset sent to the CQ was uploaded after l-g-t-m from
sandersd@chromium.org
Link to the patchset: https://codereview.chromium.org/1879253002/#ps40001
(title: "Relocate.")

[commit-bot: I haz the power, 2016-04-12 23:33:51]

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

[commit-bot: I haz the power, 2016-04-13 01:16:26]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-04-13 01:17:34]

Description was changed from

==========
Disable idle suspend for GpuVideoDecoder produced frames on OSX, Win.

The hardware decoders on these platforms can not handle being killed
while there are outstanding frames. Disable for now.

BUG=595716, 602708.
TEST=suspend does not occur on OSX, Windows.
==========

to

==========
Disable idle suspend for GpuVideoDecoder produced frames on OSX, Win.

The hardware decoders on these platforms can not handle being killed
while there are outstanding frames. Disable for now.

BUG=595716, 602708.
TEST=suspend does not occur on OSX, Windows.

Committed: https://crrev.com/7f366b224dbd21f8b8e3d93b3a4635bf68f31977
Cr-Commit-Position: refs/heads/master@{#386881}
==========

[commit-bot: I haz the power, 2016-04-13 01:17:34]

Patchset 2 (id:??) landed as
https://crrev.com/7f366b224dbd21f8b8e3d93b3a4635bf68f31977
Cr-Commit-Position: refs/heads/master@{#386881}