[sensors] Generic Sensors Framework blink side

third_party/WebKit/Source/modules/sensor/SensorReading.cpp

 // 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/SensorReading.h"
 
+#include "core/dom/ExecutionContext.h"
+#include "core/frame/LocalDOMWindow.h"
+#include "core/timing/DOMWindowPerformance.h"
+#include "core/timing/Performance.h"
+
 namespace blink {
 
-SensorReading::SensorReading()
-{
-}
-
-SensorReading::SensorReading(bool providesTimeStamp, DOMHighResTimeStamp timeStamp)
-    : m_canProvideTimeStamp(providesTimeStamp)
-    , m_timeStamp(timeStamp)
-{
-}
-
-DOMHighResTimeStamp SensorReading::timeStamp(bool& isNull)
-{
-    if (m_canProvideTimeStamp)
-        return m_timeStamp;
-
-    isNull = true;
-    return 0;
-}
-
-SensorReading::~SensorReading()
+SensorReading::SensorReading(ExecutionContext* context)
+    : m_context(context)
 {
 }
 
 DEFINE_TRACE(SensorReading)
 {
+    visitor->trace(m_sensorProxy);
+    visitor->trace(m_context);
+}
+
+void SensorReading::attach(SensorProxy* sensorProxy)
+{
+    DCHECK(sensorProxy);
+    m_sensorProxy = sensorProxy;
+}
+
+void SensorReading::detach()
+{
+    m_sensorProxy = nullptr;
+}
+
+DOMHighResTimeStamp SensorReading::timeStamp() const
+{
+    if (!m_sensorProxy)

[timvolodine, 2016/09/01 19:02:06]

nit: you could have a singe if with multiple "||"
or.. probably better: have an if with multiple conditions without the "!"
separated by "&&" and return performance->... and otherwise return 0.0 by
default.

[Mikhail, 2016/09/02 08:23:43]

Done.

+        return 0.0;
+
+    if (!m_context)
+        return 0.0;
+
+    LocalDOMWindow* window = m_context->executingWindow();
+    if (!window)
+        return 0.0;
+
+    Performance* performance = DOMWindowPerformance::performance(*window);
+    if (!performance)
+        return 0.0;
+
+    return performance->monotonicTimeToDOMHighResTimeStamp(m_sensorProxy->reading().timestamp);
 }
 
 } // namespace blink