[sensors](Linux) Fix tsan data race in sensor reader

device/generic_sensor/platform_sensor_linux.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "device/generic_sensor/platform_sensor_linux.h"
 
 #include "device/generic_sensor/linux/sensor_data_linux.h"
 #include "device/generic_sensor/platform_sensor_reader_linux.h"
 
 namespace device {
@@ skipping 11 matching lines @@
 PlatformSensorLinux::PlatformSensorLinux(
     mojom::SensorType type,
     mojo::ScopedSharedBufferMapping mapping,
     PlatformSensorProvider* provider,
     const SensorInfoLinux* sensor_device,
     scoped_refptr<base::SingleThreadTaskRunner> polling_thread_task_runner)
     : PlatformSensor(type, std::move(mapping), provider),
       default_configuration_(
           PlatformSensorConfiguration(sensor_device->device_frequency)),
       reporting_mode_(sensor_device->reporting_mode),
+      polling_thread_task_runner_(std::move(polling_thread_task_runner)),
       weak_factory_(this) {
-  sensor_reader_ =
-      SensorReader::Create(sensor_device, this, polling_thread_task_runner);
+  SensorReader::UpdateSensorReadingCallback update_reading_cb =
+      base::Bind(&PlatformSensorLinux::UpdatePlatformSensorReading,
+                 weak_factory_.GetWeakPtr());
+  SensorReader::NotifyReadErrorCallback notify_read_error_cb =
+      base::Bind(&PlatformSensorLinux::NotifyPlatformSensorError,
+                 weak_factory_.GetWeakPtr());
+
+  sensor_reader_ = SensorReader::Create(
+      sensor_device, task_runner_, update_reading_cb, notify_read_error_cb);

[Reilly Grant (use Gerrit), 2016/12/16 21:34:42]

Now you've negated the benefit of passing callbacks because they are no longer
bound to the thread. Please either:

a) Pass a TaskRunner and WeakPtr<PlatformSensorLinux>.
b) Pass 2 callbacks created with base::BindToCurrentLoop (which apparently
doesn't exist anymore?)

[maksims (do not use this acc), 2016/12/19 09:46:22]

Done.

 }
 
-PlatformSensorLinux::~PlatformSensorLinux() = default;
+PlatformSensorLinux::~PlatformSensorLinux() {
+  DCHECK(task_runner_->BelongsToCurrentThread());
+  polling_thread_task_runner_->DeleteSoon(FROM_HERE, sensor_reader_.release());
+}
 
 mojom::ReportingMode PlatformSensorLinux::GetReportingMode() {
   DCHECK(task_runner_->BelongsToCurrentThread());
   return reporting_mode_;
 }
 
 void PlatformSensorLinux::UpdatePlatformSensorReading(SensorReading reading) {
   DCHECK(task_runner_->BelongsToCurrentThread());
   bool notifyNeeded = false;
   if (GetReportingMode() == mojom::ReportingMode::ON_CHANGE) {
     if (!HaveValuesChanged(reading, old_values_))
       return;
     notifyNeeded = true;
   }
   old_values_ = reading;
   reading.timestamp = (base::TimeTicks::Now() - base::TimeTicks()).InSecondsF();
   UpdateSensorReading(reading, notifyNeeded);
 }
 
 void PlatformSensorLinux::NotifyPlatformSensorError() {
   DCHECK(task_runner_->BelongsToCurrentThread());
   NotifySensorError();
 }
 
 bool PlatformSensorLinux::StartSensor(
     const PlatformSensorConfiguration& configuration) {
   DCHECK(task_runner_->BelongsToCurrentThread());
-  if (!sensor_reader_)
-    return false;
-  return sensor_reader_->StartFetchingData(configuration);
+  polling_thread_task_runner_->PostTask(
+      FROM_HERE,
+      base::Bind(&SensorReader::StartFetchingData,
+                 base::Unretained(sensor_reader_.get()), configuration));
+  return true;
 }
 
 void PlatformSensorLinux::StopSensor() {
   DCHECK(task_runner_->BelongsToCurrentThread());
-  DCHECK(sensor_reader_);
-  sensor_reader_->StopFetchingData();
+  polling_thread_task_runner_->PostTask(
+      FROM_HERE, base::Bind(&SensorReader::StopFetchingData,
+                            base::Unretained(sensor_reader_.get())));
 }
 
 bool PlatformSensorLinux::CheckSensorConfiguration(
     const PlatformSensorConfiguration& configuration) {
   DCHECK(task_runner_->BelongsToCurrentThread());
   return configuration.frequency() > 0 &&
          configuration.frequency() <= default_configuration_.frequency();
 }
 
 PlatformSensorConfiguration PlatformSensorLinux::GetDefaultConfiguration() {
   DCHECK(task_runner_->BelongsToCurrentThread());
   return default_configuration_;
 }
 
 }  // namespace device