968603 - Add screen WakeLockListener on Windows to disable screensaver

Status: RESOLVED FIXED

(Core :: Audio/Video, defect)

Description

[Chris Pearce [:cpearce (Not reading bugmail)]]

We want the screensaver to disable during <video> playback, and WebRTC calls too.

It's pretty simple to achieve this, we need to add a nsIDOMMozWakeLockListener which disables the screen saver on "locked-foreground" state, and re-enables it otherwise.

For example, this is done for Metrofox already, using a WinRT API:
http://mxr.mozilla.org/mozilla-central/source/widget/windows/winrt/MetroAppShell.cpp#54

This bug covers Windows desktop implementation, we'll need something similar for other platforms.

There is discussion in bug 517870 as to what APIs to use on other platforms to implement this.

Comment 1

[Chris Pearce [:cpearce (Not reading bugmail)]]

Attachment: Patch

Add a wake lock listener in Win32's appshell to disable the screen saver when Gecko wants to. So we can disable the screen saver during video playback on Windows.

Comment 2

[Jim Mathies [:jimm]]

Comment on attachment 8371200 [details] [diff] [review]
Patch

Review of attachment 8371200 [details] [diff] [review]:
-----------------------------------------------------------------

I bet this would work on both platforms.

Comment 3

[Chris Pearce [:cpearce (Not reading bugmail)]]

Attachment: Patch ready for landing

Fixed a compile error discovered on Try. Patch ready for landing.

Re-trying: https://tbpl.mozilla.org/?tree=Try&rev=eed2741433c6

Comment 4

[Chris Pearce [:cpearce (Not reading bugmail)]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/85767afc495b

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/85767afc495b