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

Side by Side Diff: services/device/power_monitor/power_monitor_message_broadcaster.cc

Issue 2729293002: Fix renderers not getting power broadcast messages (Closed)
Patch Set: Created 3 years, 9 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
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "services/device/power_monitor/power_monitor_message_broadcaster.h" 5 #include "services/device/power_monitor/power_monitor_message_broadcaster.h"
6 6
7 #include "base/memory/ptr_util.h" 7 #include "base/memory/ptr_util.h"
8 #include "base/power_monitor/power_monitor.h" 8 #include "base/power_monitor/power_monitor.h"
9 #include "mojo/public/cpp/bindings/strong_binding.h" 9 #include "mojo/public/cpp/bindings/strong_binding.h"
10 10
11 namespace device { 11 namespace device {
12 12
13 PowerMonitorMessageBroadcaster::PowerMonitorMessageBroadcaster() { 13 PowerMonitorMessageBroadcaster::PowerMonitorMessageBroadcaster() {
14 base::PowerMonitor* power_monitor = base::PowerMonitor::Get(); 14 base::PowerMonitor* power_monitor = base::PowerMonitor::Get();
15 if (power_monitor) 15 if (power_monitor)
16 power_monitor->AddObserver(this); 16 power_monitor->AddObserver(this);
17 } 17 }
18 18
19 PowerMonitorMessageBroadcaster::~PowerMonitorMessageBroadcaster() { 19 PowerMonitorMessageBroadcaster::~PowerMonitorMessageBroadcaster() {
20 base::PowerMonitor* power_monitor = base::PowerMonitor::Get(); 20 base::PowerMonitor* power_monitor = base::PowerMonitor::Get();
21 if (power_monitor) 21 if (power_monitor)
22 power_monitor->RemoveObserver(this); 22 power_monitor->RemoveObserver(this);
23 } 23 }
24 24
25 // static 25 // static
26 void PowerMonitorMessageBroadcaster::Create( 26 void PowerMonitorMessageBroadcaster::Bind(
27 device::mojom::PowerMonitorRequest request) { 27 device::mojom::PowerMonitorRequest request) {
28 mojo::MakeStrongBinding(base::MakeUnique<PowerMonitorMessageBroadcaster>(), 28 bindings_.AddBinding(this, std::move(request));
29 std::move(request));
30 } 29 }
31 30
32 void PowerMonitorMessageBroadcaster::SetClient( 31 void PowerMonitorMessageBroadcaster::AddClient(
33 device::mojom::PowerMonitorClientPtr power_monitor_client) { 32 device::mojom::PowerMonitorClientPtr power_monitor_client) {
34 power_monitor_client_ = std::move(power_monitor_client); 33 clients_.AddPtr(std::move(power_monitor_client));
35 base::PowerMonitor* power_monitor = base::PowerMonitor::Get(); 34 base::PowerMonitor* power_monitor = base::PowerMonitor::Get();
36 // Unit tests does not initialize the PowerMonitor. 35 // Unit tests does not initialize the PowerMonitor.
37 if (power_monitor) 36 if (power_monitor) {
38 OnPowerStateChange(power_monitor->IsOnBatteryPower()); 37 OnPowerStateChange(power_monitor->IsOnBatteryPower());
38 }
39 } 39 }
40 40
41 void PowerMonitorMessageBroadcaster::OnPowerStateChange(bool on_battery_power) { 41 void PowerMonitorMessageBroadcaster::OnPowerStateChange(bool on_battery_power) {
42 if (power_monitor_client_) { 42 clients_.ForAllPtrs([&on_battery_power](mojom::PowerMonitorClient* client) {
43 power_monitor_client_->PowerStateChange(on_battery_power); 43 client->PowerStateChange(on_battery_power);
44 } 44 });
45 } 45 }
46 46
47 void PowerMonitorMessageBroadcaster::OnSuspend() { 47 void PowerMonitorMessageBroadcaster::OnSuspend() {
48 if (power_monitor_client_) { 48 clients_.ForAllPtrs(
49 power_monitor_client_->Suspend(); 49 [](mojom::PowerMonitorClient* client) { client->Suspend(); });
50 }
51 } 50 }
52 51
53 void PowerMonitorMessageBroadcaster::OnResume() { 52 void PowerMonitorMessageBroadcaster::OnResume() {
54 if (power_monitor_client_) { 53 clients_.ForAllPtrs(
55 power_monitor_client_->Resume(); 54 [](mojom::PowerMonitorClient* client) { client->Resume(); });
56 }
57 } 55 }
58 56
59 } // namespace device 57 } // namespace device
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698