[sensors] Introduce Generic Sensor API interfaces

device/generic_sensor/platform_sensor.h

Index: device/generic_sensor/platform_sensor.h
diff --git a/device/generic_sensor/platform_sensor.h b/device/generic_sensor/platform_sensor.h
new file mode 100644
index 0000000000000000000000000000000000000000..0bfd5bc7ae833401a98b9286ee842226e0ea92bf
--- /dev/null
+++ b/device/generic_sensor/platform_sensor.h
@@ -0,0 +1,81 @@
+// 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.
+
+#ifndef DEVICE_GENERIC_SENSOR_PLATFORM_SENSOR_H_
+#define DEVICE_GENERIC_SENSOR_PLATFORM_SENSOR_H_
+
+#include <list>
+#include <map>
+
+#include "base/macros.h"
+#include "base/memory/ref_counted.h"
+#include "base/memory/weak_ptr.h"
+#include "base/observer_list.h"
+#include "device/generic_sensor/public/interfaces/sensor.mojom.h"
+#include "mojo/public/cpp/system/buffer.h"
+
+namespace device {
+
+class PlatformSensorProvider;
+class PlatformSensorConfiguration;
+
+// Base class for the sensors provided by the platform. Concrete instances of
+// this class are created by platform specific PlatformSensorProvider.
+class PlatformSensor : public base::RefCountedThreadSafe<PlatformSensor> {
+ public:
+  // The interface that must be implemented by PlatformSensor clients.
+  class Client {
+   public:
+    virtual void OnSensorReadingChanged() = 0;
+    virtual void OnSensorError() = 0;
+    virtual bool IsNotificationSuspended() = 0;
+
+   protected:
+    virtual ~Client() {}
+  };
+
+  virtual mojom::ReportingMode GetReportingMode() = 0;
+
+  mojom::SensorType GetType() const;
+
+  bool StartListening(Client* client,
+                      const PlatformSensorConfiguration& config);
+  bool StopListening(Client* client, const PlatformSensorConfiguration& config);
+
+  void UpdateSensor();
+
+  void AddClient(Client*);
+  void RemoveClient(Client*);
+
+ protected:
+  virtual ~PlatformSensor();
+  PlatformSensor(mojom::SensorType type,
+                 mojo::ScopedSharedBufferMapping mapping,
+                 PlatformSensorProvider* provider);
+
+  using ConfigMap = std::map<Client*, std::list<PlatformSensorConfiguration>>;
+
+  virtual bool UpdateSensor(const ConfigMap& configurations) = 0;

[Reilly Grant (use Gerrit), 2016/08/11 20:59:37]

Consider naming this UpdateSensorInternal() as overloading the public function
name may be confusing.

[Mikhail, 2016/08/12 10:21:48]

Done.

+  virtual bool CheckSensorConfiguration(
+      const PlatformSensorConfiguration& configuration) = 0;
+
+  void NotifySensorReadingChanged();
+  void NotifySensorError();
+
+  mojo::ScopedSharedBufferMapping shared_buffer_mapping_;
+
+ private:
+  friend class base::RefCountedThreadSafe<PlatformSensor>;
+
+  mojom::SensorType type_;
+  base::ObserverList<Client, true> clients_;
+  ConfigMap config_map_;
+  PlatformSensorProvider* provider_;
+  base::WeakPtrFactory<PlatformSensor> weak_factory_;
+  DISALLOW_COPY_AND_ASSIGN(PlatformSensor);
+};
+
+}  // namespace device
+
+#endif  // DEVICE_GENERIC_SENSOR_PLATFORM_SENSOR_H_