Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(176)

Unified Diff: third_party/WebKit/Source/modules/sensor/AmbientLightSensorReading.cpp

Issue 2332323002: [sensors] Ambient light sensor bindings implementation (Closed)
Patch Set: Add AmbientLightSensor to the list of expected global interfaces Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/modules/sensor/AmbientLightSensorReading.cpp
diff --git a/third_party/WebKit/Source/modules/sensor/AmbientLightSensorReading.cpp b/third_party/WebKit/Source/modules/sensor/AmbientLightSensorReading.cpp
new file mode 100644
index 0000000000000000000000000000000000000000..5c923aff08c2e10cc425f1829f680551c0c8ebd3
--- /dev/null
+++ b/third_party/WebKit/Source/modules/sensor/AmbientLightSensorReading.cpp
@@ -0,0 +1,47 @@
+// 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 "modules/sensor/AmbientLightSensorReading.h"
+
+#include "modules/sensor/SensorProxy.h"
+
+namespace blink {
+
+AmbientLightSensorReading::AmbientLightSensorReading(const AmbientLightSensorReadingInit& init)
+ : SensorReading(nullptr)
+ , mAmbientLightSensorReadingInit(init)
+{
+}
+
+AmbientLightSensorReading::AmbientLightSensorReading(SensorProxy* proxy)
+ : SensorReading(proxy)
+ , mAmbientLightSensorReadingInit(AmbientLightSensorReadingInit())
+{
+}
+
+AmbientLightSensorReading::~AmbientLightSensorReading() = default;
+
+double AmbientLightSensorReading::illuminance() const
+{
+ if (mAmbientLightSensorReadingInit.hasIlluminance())
+ return mAmbientLightSensorReadingInit.illuminance();
+
+ if (!m_sensorProxy)
+ return 0.0;
timvolodine 2016/09/19 16:37:32 is this according to spec?
+ return m_sensorProxy->reading().reading[0];
+}
+
+bool AmbientLightSensorReading::isReadingUpdated(const SensorProxy::Reading& previous) const
+{
+ if (!m_sensorProxy)
+ return false;
+ return previous.reading[0] != illuminance();
+}
+
+DEFINE_TRACE(AmbientLightSensorReading)
+{
+ SensorReading::trace(visitor);
+}
+
+} // namespace blink

Powered by Google App Engine
This is Rietveld 408576698