Remove Sync GetPose VRService call, implement VRVSyncProvider

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

Index: third_party/WebKit/Source/modules/vr/VRDisplay.h
diff --git a/third_party/WebKit/Source/modules/vr/VRDisplay.h b/third_party/WebKit/Source/modules/vr/VRDisplay.h
index 8a37ac5c821830a7dd5fedd78983876ca20c51da..0456f10e66d1d00d7291aca009d472bde8ef6c05 100644
--- a/third_party/WebKit/Source/modules/vr/VRDisplay.h
+++ b/third_party/WebKit/Source/modules/vr/VRDisplay.h
@@ -7,6 +7,7 @@
 
 #include "core/dom/Document.h"
 #include "core/dom/FrameRequestCallback.h"
+#include "core/dom/TaskRunnerHelper.h"

[haraken, 2017/01/19 00:49:56]

Not needed?

[mthiesse, 2017/01/19 00:54:04]

Done.

 #include "core/events/EventTarget.h"
 #include "device/vr/vr_service.mojom-blink.h"
 #include "modules/vr/VRDisplayCapabilities.h"
@@ -72,7 +73,6 @@ class VRDisplay final : public EventTargetWithInlineData,
 
   int requestAnimationFrame(FrameRequestCallback*);
   void cancelAnimationFrame(int id);
-  void serviceScriptedAnimations(double monotonicAnimationStartTime);
 
   ScriptPromise requestPresent(ScriptState*, const HeapVector<VRLayer>& layers);
   ScriptPromise exitPresent(ScriptState*);
@@ -131,41 +131,46 @@ class VRDisplay final : public EventTargetWithInlineData,
   void OnActivate(device::mojom::blink::VRDisplayEventReason) override;
   void OnDeactivate(device::mojom::blink::VRDisplayEventReason) override;
 
+  void OnVSync(device::mojom::blink::VRPosePtr,
+               mojo::common::mojom::blink::TimeDeltaPtr);
+  void ConnectVSyncProvider();
+
   ScriptedAnimationController& ensureScriptedAnimationController(Document*);
 
   Member<NavigatorVR> m_navigatorVR;
-  unsigned m_displayId;
+  unsigned m_displayId = 0;
   String m_displayName;
-  bool m_isConnected;
-  bool m_isPresenting;
-  bool m_isValidDeviceForPresenting;
-  bool m_canUpdateFramePose;
+  bool m_isConnected = false;
+  bool m_isPresenting = false;
+  bool m_isValidDeviceForPresenting = true;
   Member<VRDisplayCapabilities> m_capabilities;
   Member<VRStageParameters> m_stageParameters;
   Member<VREyeParameters> m_eyeParametersLeft;
   Member<VREyeParameters> m_eyeParametersRight;
   device::mojom::blink::VRPosePtr m_framePose;
   VRLayer m_layer;
-  double m_depthNear;
-  double m_depthFar;
+  double m_depthNear = 0.01;
+  double m_depthFar = 10000.0;
 
   void dispose();
 
   Timer<VRDisplay> m_fullscreenCheckTimer;
   String m_fullscreenOrigWidth;
   String m_fullscreenOrigHeight;
-  gpu::gles2::GLES2Interface* m_contextGL;
+  gpu::gles2::GLES2Interface* m_contextGL = nullptr;
   Member<WebGLRenderingContextBase> m_renderingContext;
 
   Member<ScriptedAnimationController> m_scriptedAnimationController;
-  bool m_animationCallbackRequested;
-  bool m_inAnimationFrame;
-  bool m_displayBlurred;
-  bool m_reenteredFullscreen;
+  bool m_pendingRaf = false;
+  bool m_inAnimationFrame = false;
+  bool m_displayBlurred = false;
+  bool m_reenteredFullscreen = false;
+  double m_timebase = -1;
 
   device::mojom::blink::VRDisplayPtr m_display;
 
-  mojo::Binding<device::mojom::blink::VRDisplayClient> m_binding;
+  mojo::Binding<device::mojom::blink::VRDisplayClient> m_displayClientBinding;
+  device::mojom::blink::VRVSyncProviderPtr m_vrVSyncProvider;
 
   HeapDeque<Member<ScriptPromiseResolver>> m_pendingPresentResolvers;
 };