[sensors] [mac] Implement ambient light sensor for macOS

device/generic_sensor/platform_sensor_ambient_light_mac.h

Index: device/generic_sensor/platform_sensor_ambient_light_mac.h
diff --git a/device/generic_sensor/platform_sensor_ambient_light_mac.h b/device/generic_sensor/platform_sensor_ambient_light_mac.h
new file mode 100644
index 0000000000000000000000000000000000000000..68a3deab2429789b26d096ea4433050a055dcaca
--- /dev/null
+++ b/device/generic_sensor/platform_sensor_ambient_light_mac.h
@@ -0,0 +1,57 @@
+// 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_PLATFORM_SENSOR_AMBIENT_LIGHT_SENSOR_MAC_H_
+#define DEVICE_GENERIC_PLATFORM_SENSOR_AMBIENT_LIGHT_SENSOR_MAC_H_
+
+#include <IOKit/IOKitLib.h>
+
+#include "base/mac/scoped_ioobject.h"
+#include "device/generic_sensor/platform_sensor.h"
+
+namespace base {
+class SingleThreadTaskRunner;
+}
+
+namespace device {
+
+class PlatformSensorAmbientLightMac : public PlatformSensor {

[Robert Sesek, 2016/10/05 21:00:18]

This class should have some documentation. Especially the ctor with the number
of arguments it takes.

+ public:
+  PlatformSensorAmbientLightMac(
+      mojom::SensorType type,
+      mojo::ScopedSharedBufferMapping mapping,
+      uint64_t buffer_size,
+      PlatformSensorProvider* provider,
+      scoped_refptr<base::SingleThreadTaskRunner> task_runner);
+
+  mojom::ReportingMode GetReportingMode() override;
+  bool StartSensor(const PlatformSensorConfiguration& configuration) override;
+  void StopSensor() override;
+
+ protected:
+  ~PlatformSensorAmbientLightMac() override;
+  bool CheckSensorConfiguration(
+      const PlatformSensorConfiguration& configuration) override;
+  PlatformSensorConfiguration GetDefaultConfiguration() override;
+  void UpdateReading(uint64_t lux_values[2]);
+
+ private:
+  static void IOServiceCallback(void* context,
+                                io_service_t service,
+                                natural_t message_type,
+                                void*);

[Robert Sesek, 2016/10/05 21:00:18]

name here too

+
+  base::mac::ScopedIOObject<io_service_t> light_sensor_service_;

[Robert Sesek, 2016/10/05 21:00:18]

And these fields should be documented too.

+  IONotificationPortRef light_sensor_port_;
+  base::mac::ScopedIOObject<io_object_t> light_sensor_object_;
+  base::mac::ScopedIOObject<io_object_t> light_sensor_notification_;
+  scoped_refptr<base::SingleThreadTaskRunner> task_runner_;
+  double current_lux_;
+
+  DISALLOW_COPY_AND_ASSIGN(PlatformSensorAmbientLightMac);
+};
+
+}  // namespace device
+
+#endif  // DEVICE_GENERIC_PLATFORM_SENSOR_AMBIENT_LIGHT_SENSOR_MAC_H_