[Sensors] Implement sensor data polling

third_party/WebKit/Source/modules/sensor/SensorProviderProxy.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/SensorProviderProxy.h"
 
 #include "modules/sensor/SensorProxy.h"
 #include "platform/mojo/MojoHelper.h"
 #include "public/platform/InterfaceProvider.h"
 
@@ skipping 21 matching lines @@
     }
     return result;
 }
 
 SensorProviderProxy::~SensorProviderProxy()
 {
 }
 
 DEFINE_TRACE(SensorProviderProxy)
 {
+    visitor->trace(m_frame);
     visitor->trace(m_sensors);
     Supplement<LocalFrame>::trace(visitor);
 }
 
 SensorProxy* SensorProviderProxy::getOrCreateSensor(device::mojom::blink::SensorType type)
 {
     for (SensorProxy* sensor : m_sensors) {
         // TODO(Mikhail) : Hash sensors by type for efficiency.
         if (sensor->type() == type)
             return sensor;
     }
 
-    SensorProxy* sensor = new SensorProxy(type, this);
+    SensorProxy* sensor = new SensorProxy(type, m_frame->page(), this);

[haraken, 2016/09/08 11:07:13]

How is this working? It looks like that no one is assigning a valid frame to
m_frame. So m_frame should be nullptr here.

[Mikhail, 2016/09/08 11:24:48]

Ah I missed m_frame initialization when split the code, sorry for that

     m_sensors.add(sensor);
 
     return sensor;
 }
 
 void SensorProviderProxy::onSensorProviderConnectionError()
 {
     m_sensorProvider.reset();
     for (SensorProxy* sensor : m_sensors)
         sensor->handleSensorError();
 }
 
 } // namespace blink