[Cast, Android] Don't hold the wake lock if the video is playing remotely.

[Cast, Android] Don't hold the wake lock if the video is playing remotely.

Since the video is actually playing on a remote device, there's no need to prevent
the screen from fading and locking.

BUG=420665
TEST=manual

Committed: https://crrev.com/68017be636f8e3446174f9915503fb80345bf54f
Cr-Commit-Position: refs/heads/master@{#299137}

[whywhat, 2014-10-08 21:00:21]

avayvod@chromium.org changed reviewers:
+ nasko@chromium.org, qinmin@chromium.org

[nasko, 2014-10-08 23:53:22]

LGTM

[qinmin, 2014-10-09 01:18:15]

what will happen if video is playing locally with wake lock acquired, then
casted to remote device?

In that case, will MediaPlayingNotification gets sent? That will require a
pause()/play() on the media element when doing casting, but I didn't see that in
the code.

[whywhat, 2014-10-09 09:21:34]

On 2014/10/09 01:18:15, qinmin wrote:
> what will happen if video is playing locally with wake lock acquired, then
> casted to remote device?
> 
> In that case, will MediaPlayingNotification gets sent? That will require a
> pause()/play() on the media element when doing casting, but I didn't see that
in
> the code.

We already do pause the local player and send the pause IPC to the
webmediaplayer before we create a remote media player. Then the playing command
is sent for the remote player.

[qinmin, 2014-10-10 17:03:32]

lgtm

[whywhat, 2014-10-10 17:33:48]

The CQ bit was checked by avayvod@chromium.org

[commit-bot: I haz the power, 2014-10-10 17:34:48]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/644463003/330001

[commit-bot: I haz the power, 2014-10-10 18:02:55]

Committed patchset #5 (id:330001)

[commit-bot: I haz the power, 2014-10-10 18:03:40]

Patchset 5 (id:??) landed as
https://crrev.com/68017be636f8e3446174f9915503fb80345bf54f
Cr-Commit-Position: refs/heads/master@{#299137}