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

device/generic_sensor/platform_sensor_linux.cc

Index: device/generic_sensor/platform_sensor_linux.cc
diff --git a/device/generic_sensor/platform_sensor_linux.cc b/device/generic_sensor/platform_sensor_linux.cc
index 55aa1398d1499c7bd61bfc4ec10b1199b5087c85..df6e22ba19377efb9f350d0fc6ac0b38c48ca244 100644
--- a/device/generic_sensor/platform_sensor_linux.cc
+++ b/device/generic_sensor/platform_sensor_linux.cc
@@ -4,6 +4,7 @@
 
 #include "device/generic_sensor/platform_sensor_linux.h"
 
+#include "base/bind_to_current_loop.h"
 #include "device/generic_sensor/linux/sensor_data_linux.h"
 #include "device/generic_sensor/platform_sensor_reader_linux.h"
 
@@ -29,12 +30,25 @@ PlatformSensorLinux::PlatformSensorLinux(
       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::BindToCurrentLoop(
+          base::Bind(&PlatformSensorLinux::UpdatePlatformSensorReading,
+                     weak_factory_.GetWeakPtr()));
+  SensorReader::NotifyReadErrorCallback notify_read_error_cb =
+      base::BindToCurrentLoop(
+          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/15 18:51:31]

Since these callbacks are bound to the current message loop the task_runner_
passed to SensorReader::Create is unused.

Like my earlier comments, since SensorReader will only ever call these two
functions on PlatformSensorLinux I would rather we didn't use callbacks here and
just passed task_runner_ and weak_factory_.GetWeakPtr() to SensorReader::Create
but either way works fine.

[maksims (do not use this acc), 2016/12/16 08:39:22]

I think passing a callback with binded weakPtr to a sensor and a task runner is
our solution.

 }
 
-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());
@@ -62,15 +76,18 @@ void PlatformSensorLinux::NotifyPlatformSensorError() {
 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(