diff options
| author | jbauch <jbauch@webrtc.org> | 2015-08-07 06:39:21 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-08-07 13:39:56 +0000 |
| commit | 6571bc9c6dbc7486326f26292a5e351b938d18a7 (patch) | |
| tree | f80d60b4ec87fea92690406d4491b3803ffe7b93 /device/battery | |
| parent | b18b341ddae2051da721b9ddf40b884e44819910 (diff) | |
| download | chromium_src-6571bc9c6dbc7486326f26292a5e351b938d18a7.zip chromium_src-6571bc9c6dbc7486326f26292a5e351b938d18a7.tar.gz chromium_src-6571bc9c6dbc7486326f26292a5e351b938d18a7.tar.bz2 | |
Fix non-working battery status manager on Linux.
The change in https://crrev.com/2438a3e2acb1318a4e59c78764b532bac3649161
removed the call to start the notification thread and instead always returned
"false".
This CL fixes this by starting the thread (and thus actually getting status
changes).
BUG=420623
R=timvolodine@chromium.org
Review URL: https://codereview.chromium.org/1263353004
Cr-Commit-Position: refs/heads/master@{#342343}
Diffstat (limited to 'device/battery')
| -rw-r--r-- | device/battery/battery_status_manager_linux.cc | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/device/battery/battery_status_manager_linux.cc b/device/battery/battery_status_manager_linux.cc index 7fdd86f..0b2da63 100644 --- a/device/battery/battery_status_manager_linux.cc +++ b/device/battery/battery_status_manager_linux.cc @@ -274,6 +274,7 @@ class BatteryStatusManagerLinux : public BatteryStatusManager { private: // BatteryStatusManager: bool StartListeningBatteryChange() override { + if (!StartNotifierThreadIfNecessary()) return false; notifier_thread_->message_loop()->PostTask( |