Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(208)

Side by Side Diff: remoting/host/host_power_save_blocker.cc

Issue 2080723008: [Chromoting] Use device::PowerSaveBlocker to block screen saver (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Resolve review comments Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "remoting/host/host_power_save_blocker.h"
6
7 #include <utility>
8
9 #include "base/logging.h"
10 #include "remoting/base/auto_thread_task_runner.h"
11 #include "remoting/host/host_status_monitor.h"
12
13 namespace remoting {
14
15 HostPowerSaveBlocker::HostPowerSaveBlocker(
16 base::WeakPtr<HostStatusMonitor> monitor,
17 const ChromotingHostContext& context)
Sergey Ulanov 2016/07/13 04:57:05 Please replace this parameter with two task runner
Hzj_jie 2016/07/13 20:27:13 Done.
18 : monitor_(monitor),
19 ui_task_runner_(context.ui_task_runner()),
20 file_task_runner_(context.file_task_runner()) {
21 DCHECK(monitor_);
22 monitor_->AddStatusObserver(this);
23 }
24
25 HostPowerSaveBlocker::~HostPowerSaveBlocker() {
26 if (monitor_.get()) {
27 monitor_->RemoveStatusObserver(this);
28 }
29 }
30
31 void HostPowerSaveBlocker::OnClientConnected(const std::string& jid) {
32 blocker_.reset(new device::PowerSaveBlocker(
33 device::PowerSaveBlocker::kPowerSaveBlockPreventDisplaySleep,
34 device::PowerSaveBlocker::kReasonOther,
35 "Remoting session is active",
36 ui_task_runner_,
37 file_task_runner_));
38 }
39
40 void HostPowerSaveBlocker::OnClientDisconnected(const std::string& jid) {
41 blocker_.reset();
42 }
43
44 } // namespace remoting
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698