[OnionSoup] Moving VR service from content to Blink

third_party/WebKit/Source/modules/vr/VRController.h

 // Copyright 2015 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 VRController_h
 #define VRController_h
 
 #include "core/frame/LocalFrameLifecycleObserver.h"
 #include "modules/ModulesExport.h"
+#include "modules/vr/VRTypeConverters.h"
 #include "platform/Supplementable.h"
-#include "public/platform/modules/vr/WebVR.h"
-#include "public/platform/modules/vr/WebVRClient.h"
+#include "wtf/Deque.h"
+#include "wtf/PassOwnPtr.h"
 
 namespace blink {
 
+class ServiceRegistry;
+class VRGetDevicesCallback;
+
 class MODULES_EXPORT VRController final
     : public GarbageCollectedFinalized<VRController>
     , public Supplement<LocalFrame>
     , public LocalFrameLifecycleObserver {
     USING_GARBAGE_COLLECTED_MIXIN(VRController);
     WTF_MAKE_NONCOPYABLE(VRController);
 public:
     virtual ~VRController();
 
-    void getDevices(WebVRGetDevicesCallback*);
+    void getDevices(PassOwnPtr<VRGetDevicesCallback>);

[dcheng, 2016/04/28 19:59:44]

Use std::unique_ptr in new code, here and elsewhere.

[RaviKasibhatla, 2016/04/29 14:49:05]

IIRC, @yutak recently sent a broadcast that the transition work for using
std::unique_ptr in place of OwnPtr is ready to land. But he also mentioned that
there is next step of work which will make OwnPtr behavior as close as possible
to std::unique_ptr.
Shouldn't we transition to unique_ptr after announcement from yutak that it is
good to go?

[dcheng, 2016/04/29 17:28:11]

No, new code should just use std::unique_ptr. It will save us migration work
down the road.

[RaviKasibhatla, 2016/05/03 14:34:21]

Done.

 
     void getSensorState(unsigned index, WebHMDSensorState& into);
 
     void resetSensor(unsigned index);
 
-    static void provideTo(LocalFrame&, WebVRClient*);
+    static void provideTo(LocalFrame&, ServiceRegistry*);
     static VRController* from(LocalFrame&);
     static const char* supplementName();
 
     DECLARE_VIRTUAL_TRACE();
 
 private:
-    VRController(LocalFrame&, WebVRClient*);
+    VRController(LocalFrame&, ServiceRegistry*);
 
     // Inherited from LocalFrameLifecycleObserver.
     void willDetachFrameHost() override;
 
-    WebVRClient* m_client;
+    // Binding callbacks.
+    void OnGetDevices(const mojo::Array<mojom::VRDeviceInfoPtr>&);
+    void OnGetSensorState(const mojom::VRSensorStatePtr&);
+
+    WTF::Deque<OwnPtr<VRGetDevicesCallback>> m_pendingGetDevicesCallbacks;
+    // SensorState is returned synchronously. No need for storing multiple requests.
+    WebHMDSensorState* m_pendingSensorStateRequest;

[haraken, 2016/04/28 14:50:03]

Why does this need to be a pointer?

[RaviKasibhatla, 2016/04/28 18:08:33]

getSensorState() passes a reference to WebHMDSensorState variable which is
expected to be populated. Taking a pointer to populate the same address
variable, as passed by the callee of getSensorState() API.

[haraken, 2016/04/28 19:16:15]

How can the caller of getSensorState() know the timing when the pointer value is
updated?

+    mojom::VRServicePtr m_service;
 };
 
 } // namespace blink
 
 #endif // VRController_h