Index: device/generic_sensor/platform_sensor_provider_base.cc |
diff --git a/device/generic_sensor/platform_sensor_provider_base.cc b/device/generic_sensor/platform_sensor_provider_base.cc |
index f8bb907f0f27f013da38c99a13a329bf7ad0af20..27abaa451840519fea7105484535bd17b587c878 100644 |
--- a/device/generic_sensor/platform_sensor_provider_base.cc |
+++ b/device/generic_sensor/platform_sensor_provider_base.cc |
@@ -19,13 +19,18 @@ const uint64_t kSharedBufferSizeInBytes = |
} // namespace |
-PlatformSensorProviderBase::PlatformSensorProviderBase() = default; |
-PlatformSensorProviderBase::~PlatformSensorProviderBase() = default; |
+PlatformSensorProviderBase::PlatformSensorProviderBase() { |
+ DETACH_FROM_THREAD(thread_checker_); |
Reilly Grant (use Gerrit)
2017/05/15 15:23:35
No need to add a DETACH_FROM_THREAD here.
ke.he
2017/05/16 03:09:22
Done.
|
+} |
+ |
+PlatformSensorProviderBase::~PlatformSensorProviderBase() { |
+ DCHECK_CALLED_ON_VALID_THREAD(thread_checker_); |
+} |
void PlatformSensorProviderBase::CreateSensor( |
mojom::SensorType type, |
const CreateSensorCallback& callback) { |
- DCHECK(CalledOnValidThread()); |
+ DCHECK_CALLED_ON_VALID_THREAD(thread_checker_); |
if (!CreateSharedBufferIfNeeded()) { |
callback.Run(nullptr); |
@@ -53,7 +58,7 @@ void PlatformSensorProviderBase::CreateSensor( |
scoped_refptr<PlatformSensor> PlatformSensorProviderBase::GetSensor( |
mojom::SensorType type) { |
- DCHECK(CalledOnValidThread()); |
+ DCHECK_CALLED_ON_VALID_THREAD(thread_checker_); |
auto it = sensor_map_.find(type); |
if (it != sensor_map_.end()) |
@@ -62,7 +67,7 @@ scoped_refptr<PlatformSensor> PlatformSensorProviderBase::GetSensor( |
} |
bool PlatformSensorProviderBase::CreateSharedBufferIfNeeded() { |
- DCHECK(CalledOnValidThread()); |
+ DCHECK_CALLED_ON_VALID_THREAD(thread_checker_); |
if (shared_buffer_handle_.is_valid()) |
return true; |
@@ -72,7 +77,7 @@ bool PlatformSensorProviderBase::CreateSharedBufferIfNeeded() { |
} |
void PlatformSensorProviderBase::RemoveSensor(mojom::SensorType type) { |
- DCHECK(CalledOnValidThread()); |
+ DCHECK_CALLED_ON_VALID_THREAD(thread_checker_); |
DCHECK(ContainsKey(sensor_map_, type)); |
sensor_map_.erase(type); |
@@ -84,21 +89,21 @@ void PlatformSensorProviderBase::RemoveSensor(mojom::SensorType type) { |
mojo::ScopedSharedBufferHandle |
PlatformSensorProviderBase::CloneSharedBufferHandle() { |
- DCHECK(CalledOnValidThread()); |
+ DCHECK_CALLED_ON_VALID_THREAD(thread_checker_); |
CreateSharedBufferIfNeeded(); |
return shared_buffer_handle_->Clone( |
mojo::SharedBufferHandle::AccessMode::READ_ONLY); |
} |
bool PlatformSensorProviderBase::HasSensors() const { |
- DCHECK(CalledOnValidThread()); |
+ DCHECK_CALLED_ON_VALID_THREAD(thread_checker_); |
return !sensor_map_.empty(); |
} |
void PlatformSensorProviderBase::NotifySensorCreated( |
mojom::SensorType type, |
scoped_refptr<PlatformSensor> sensor) { |
- DCHECK(CalledOnValidThread()); |
+ DCHECK_CALLED_ON_VALID_THREAD(thread_checker_); |
DCHECK(!ContainsKey(sensor_map_, type)); |
DCHECK(ContainsKey(requests_map_, type)); |