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

Side by Side Diff: chromeos/accelerometer/accelerometer_reader.cc

Issue 934843002: Implement DeviceMotionEvent API (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Update Thread Safety Created 5 years, 10 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 "chromeos/accelerometer/accelerometer_reader.h" 5 #include "chromeos/accelerometer/accelerometer_reader.h"
6 6
7 #include <string> 7 #include <string>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/files/file_util.h" 10 #include "base/files/file_util.h"
(...skipping 184 matching lines...) Expand 10 before | Expand all | Expand 10 after
195 // Asynchronously detect and initialize the accelerometer to avoid delaying 195 // Asynchronously detect and initialize the accelerometer to avoid delaying
196 // startup. 196 // startup.
197 base::PostTaskAndReplyWithResult( 197 base::PostTaskAndReplyWithResult(
198 task_runner_.get(), FROM_HERE, 198 task_runner_.get(), FROM_HERE,
199 base::Bind(&DetectAndReadAccelerometerConfiguration, configuration_), 199 base::Bind(&DetectAndReadAccelerometerConfiguration, configuration_),
200 base::Bind(&AccelerometerReader::OnInitialized, 200 base::Bind(&AccelerometerReader::OnInitialized,
201 weak_factory_.GetWeakPtr(), configuration_)); 201 weak_factory_.GetWeakPtr(), configuration_));
202 } 202 }
203 203
204 void AccelerometerReader::AddObserver(Observer* observer) { 204 void AccelerometerReader::AddObserver(Observer* observer) {
205 observers_.AddObserver(observer); 205 observers_->AddObserver(observer);
206 if (has_update_) 206 if (has_update_)
207 observer->OnAccelerometerUpdated(update_); 207 observer->OnAccelerometerUpdated(update_);
208 } 208 }
209 209
210 void AccelerometerReader::RemoveObserver(Observer* observer) { 210 void AccelerometerReader::RemoveObserver(Observer* observer) {
211 observers_.RemoveObserver(observer); 211 observers_->RemoveObserver(observer);
212 }
213
214 bool AccelerometerReader::HasObserver(Observer* observer) {
215 return observers_->HasObserver(observer);
212 } 216 }
213 217
214 AccelerometerReader::AccelerometerReader() 218 AccelerometerReader::AccelerometerReader()
215 : has_update_(false), 219 : has_update_(false),
216 configuration_(new AccelerometerReader::Configuration()), 220 configuration_(new AccelerometerReader::Configuration()),
221 observers_(new ObserverListThreadSafe<Observer>()),
217 weak_factory_(this) { 222 weak_factory_(this) {
218 } 223 }
219 224
220 AccelerometerReader::~AccelerometerReader() { 225 AccelerometerReader::~AccelerometerReader() {
221 } 226 }
222 227
223 void AccelerometerReader::OnInitialized( 228 void AccelerometerReader::OnInitialized(
224 scoped_refptr<AccelerometerReader::Configuration> configuration, 229 scoped_refptr<AccelerometerReader::Configuration> configuration,
225 bool success) { 230 bool success) {
226 if (success) 231 if (success)
(...skipping 27 matching lines...) Expand all
254 259
255 int16* values = reinterpret_cast<int16*>(reading->data); 260 int16* values = reinterpret_cast<int16*>(reading->data);
256 update_.Set(static_cast<AccelerometerSource>(i), 261 update_.Set(static_cast<AccelerometerSource>(i),
257 values[configuration_->data.index[i][0]] * 262 values[configuration_->data.index[i][0]] *
258 configuration_->data.scale[i][0], 263 configuration_->data.scale[i][0],
259 values[configuration_->data.index[i][1]] * 264 values[configuration_->data.index[i][1]] *
260 configuration_->data.scale[i][1], 265 configuration_->data.scale[i][1],
261 values[configuration_->data.index[i][2]] * 266 values[configuration_->data.index[i][2]] *
262 configuration_->data.scale[i][2]); 267 configuration_->data.scale[i][2]);
263 } 268 }
264 FOR_EACH_OBSERVER(Observer, observers_, OnAccelerometerUpdated(update_)); 269 observers_->Notify(FROM_HERE, &Observer::OnAccelerometerUpdated, update_);
flackr 2015/02/25 23:15:16 nit: Add TODO to move this to blocking thread (ref
jonross 2015/03/02 15:18:10 Done.
265 } 270 }
266 271
267 // Trigger another read after the current sampling delay. 272 // Trigger another read after the current sampling delay.
268 base::MessageLoop::current()->PostDelayedTask( 273 base::MessageLoop::current()->PostDelayedTask(
269 FROM_HERE, 274 FROM_HERE,
270 base::Bind(&AccelerometerReader::TriggerRead, 275 base::Bind(&AccelerometerReader::TriggerRead,
271 weak_factory_.GetWeakPtr()), 276 weak_factory_.GetWeakPtr()),
272 base::TimeDelta::FromMilliseconds(kDelayBetweenReadsMs)); 277 base::TimeDelta::FromMilliseconds(kDelayBetweenReadsMs));
273 } 278 }
274 279
275 } // namespace chromeos 280 } // namespace chromeos
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698