Assert both display and system keep-alive on each capture.

Assert both display and system keep-alive on each capture.

This ensures that while the screen is actively being captured,
the host system will not suspend, e.g. due to inactivity.

BUG=129912
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=181864

[Wez, 2013-02-09 02:17:16]

PTAL

[alexeypa (please no reviews), 2013-02-11 17:02:28]

lgtm

https://codereview.chromium.org/12211101/diff/1/media/video/capture/screen/sc...
File media/video/capture/screen/screen_capturer_win.cc (right):

https://codereview.chromium.org/12211101/diff/1/media/video/capture/screen/sc...
media/video/capture/screen/screen_capturer_win.cc:221: LOG(WARNING) << "Failed
to make system & display power assertion";
nit #1: LOG_GETLASTERROR(WARNING).

nit #2: this will flood the log if SetThreadExecutionState() fails all the time.
Add some rate limiting logic. For instance do not log the _same_ error unless X
seconds passed since the last one.

[Wez, 2013-02-12 01:52:46]

https://codereview.chromium.org/12211101/diff/1/media/video/capture/screen/sc...
File media/video/capture/screen/screen_capturer_win.cc (right):

https://codereview.chromium.org/12211101/diff/1/media/video/capture/screen/sc...
media/video/capture/screen/screen_capturer_win.cc:221: LOG(WARNING) << "Failed
to make system & display power assertion";
On 2013/02/11 17:02:28, alexeypa wrote:
> nit #1: LOG_GETLASTERROR(WARNING).
> 
> nit #2: this will flood the log if SetThreadExecutionState() fails all the
time.
> Add some rate limiting logic. For instance do not log the _same_ error unless
X
> seconds passed since the last one. 

I've settled on just logging the first failure per capturer, which should be
enough for the main debug value of determining why the user's machine fell
asleep. :)

[commit-bot: I haz the power, 2013-02-12 01:55:35]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/wez@chromium.org/12211101/2002

[commit-bot: I haz the power, 2013-02-12 05:08:49]

Change committed as 181864