Prevent the screen from sleeping when the desktop is being captured.

Prevent the screen from sleeping when the desktop is being captured.

BUG=405784
Committed: https://crrev.com/aea4a57a952865dca22498a92c1abafa13d6507a
Cr-Commit-Position: refs/heads/master@{#291908}

[jiayl, 2014-08-25 19:13:27]

jiayl@chromium.org changed reviewers:
+ sergeyu@chromium.org

[jiayl, 2014-08-25 19:13:27]

Sergey,

PTAL.

[juberti2, 2014-08-25 19:16:24]

juberti@chromium.org changed reviewers:
+ juberti@chromium.org

[juberti2, 2014-08-25 19:16:24]

https://codereview.chromium.org/502103003/diff/1/content/browser/media/captur...
File content/browser/media/capture/desktop_capture_device.cc (right):

https://codereview.chromium.org/502103003/diff/1/content/browser/media/captur...
content/browser/media/capture/desktop_capture_device.cc:172:
"DesktopCatureDevice is running").release());
DesktopCatureDevice -> DesktopCaptureDevice (here and other file)

[jiayl, 2014-08-25 19:50:01]

https://codereview.chromium.org/502103003/diff/1/content/browser/media/captur...
File content/browser/media/capture/desktop_capture_device.cc (right):

https://codereview.chromium.org/502103003/diff/1/content/browser/media/captur...
content/browser/media/capture/desktop_capture_device.cc:172:
"DesktopCatureDevice is running").release());
On 2014/08/25 19:16:24, juberti2 wrote:
> DesktopCatureDevice -> DesktopCaptureDevice (here and other file)

Done.

[Sergey Ulanov, 2014-08-25 23:45:10]

I'd prefer treating this as a temporary solution until we have a proper solution
using a separate web or extension API to keep machine awake.
So maybe add TODO and open a bug to revert this change later?
LGTM once TODO is added.

[juberti2, 2014-08-25 23:46:39]

On 2014/08/25 23:45:10, Sergey Ulanov wrote:
> I'd prefer treating this as a temporary solution until we have a proper
solution
> using a separate web or extension API to keep machine awake.
> So maybe add TODO and open a bug to revert this change later?
> LGTM once TODO is added.

lgtm, although any todo should apply to window and screen capturers.

[jiayl, 2014-08-25 23:51:02]

The CQ bit was checked by jiayl@chromium.org

[commit-bot: I haz the power, 2014-08-25 23:53:14]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/jiayl@chromium.org/502103003/40001

[Sergey Ulanov, 2014-08-25 23:55:31]

On 2014/08/25 23:46:39, juberti2 wrote:
> On 2014/08/25 23:45:10, Sergey Ulanov wrote:
> > I'd prefer treating this as a temporary solution until we have a proper
> solution
> > using a separate web or extension API to keep machine awake.
> > So maybe add TODO and open a bug to revert this change later?
> > LGTM once TODO is added.
> 
> lgtm, although any todo should apply to window and screen capturers.

I agree. The fact that we had this feature for screen capturers is just side
effect of sharing this code with CRD Host.

[commit-bot: I haz the power, 2014-08-26 02:43:42]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-08-26 02:43:44]

Failed to commit the patch.

[jiayl, 2014-08-26 15:31:00]

The CQ bit was checked by jiayl@chromium.org

[commit-bot: I haz the power, 2014-08-26 15:31:50]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/jiayl@chromium.org/502103003/40001

[commit-bot: I haz the power, 2014-08-26 15:34:14]

Committed patchset #3 (40001) as 617f203d9ff1d06b45e45613533b2aacbfe1a76c

[commit-bot: I haz the power, 2014-09-10 02:42:59]

Patchset 3 (id:??) landed as
https://crrev.com/aea4a57a952865dca22498a92c1abafa13d6507a
Cr-Commit-Position: refs/heads/master@{#291908}