Chromium Code Reviews| Index: media/blink/webmediaplayer_impl.cc |
| diff --git a/media/blink/webmediaplayer_impl.cc b/media/blink/webmediaplayer_impl.cc |
| index e5d8753edfd74d85943784112eba8da26f218d6d..ea2683471d76910e58120dc6a612632f794e95b6 100644 |
| --- a/media/blink/webmediaplayer_impl.cc |
| +++ b/media/blink/webmediaplayer_impl.cc |
| @@ -1452,10 +1452,11 @@ void WebMediaPlayerImpl::SetSuspendState(bool is_suspended) { |
| if (IsNetworkStateError(network_state_)) |
| return; |
| -#if defined(OS_MACOSX) || defined(OS_WIN) |
| - // TODO(sandersd): Idle suspend is disabled on OSX and Windows for hardware |
| - // decoding / opaque video frames since these frames are owned by the decoder |
| - // in the GPU process. http://crbug.com/595716 and http://crbug.com/602708 |
| +#if !defined(OS_ANDROID) |
|
DaleCurtis
2016/05/28 00:23:32
OS_ANDROID, OS_CHROMEOS are worth keeping I think.
|
| + // TODO(sandersd): idle suspend is disabled if decoder owns video frame: used |
| + // on OSX+Windows hardware decoding/opaque video frames (decoder in GPU |
| + // process) and on Chromecast (decoder in browser process). |
| + // http://crbug.com/595716 and http://crbug.com/602708 |
| if (can_suspend_state_ == CanSuspendState::UNKNOWN) { |
| scoped_refptr<VideoFrame> frame = GetCurrentFrameFromCompositor(); |
| if (frame) { |