mojo VR interface simplified

device/vr/test/fake_vr_device.h

Index: device/vr/test/fake_vr_device.h
diff --git a/device/vr/test/fake_vr_device.h b/device/vr/test/fake_vr_device.h
index a72a1941003e99c282a9e9550955aef901166e95..0e1f2ce1be997ad94a9b48752c0064c9273cf296 100644
--- a/device/vr/test/fake_vr_device.h
+++ b/device/vr/test/fake_vr_device.h
@@ -18,18 +18,28 @@ class FakeVRDevice : public VRDevice {
 
   void InitBasicDevice();
 
-  void SetVRDevice(const VRDisplayPtr& device);
-  void SetPose(const VRPosePtr& state);
+  void SetVRDevice(const mojom::VRDisplayInfoPtr& device);
+  void SetPose(const mojom::VRPosePtr& state);
+  unsigned UpdatedNum() { return updated_; }

[dcheng, 2016/11/08 08:55:16]

Nit: const

 
-  VRDisplayPtr GetVRDevice() override;
-  VRPosePtr GetPose() override;
-  void ResetPose() override;
+  mojom::VRDisplayInfoPtr GetVRDevice() override;
+  mojom::VRPosePtr GetPose(VRServiceImpl* service) override;
+  void ResetPose(VRServiceImpl* service) override;
+
+  bool RequestPresent(VRServiceImpl* service, bool secure_origin) override;
+  void ExitPresent(VRServiceImpl* service) override;
+  void SubmitFrame(VRServiceImpl* service, mojom::VRPosePtr pose) override;
+  void UpdateLayerBounds(VRServiceImpl* service,
+                         mojom::VRLayerBoundsPtr leftBounds,
+                         mojom::VRLayerBoundsPtr rightBounds) override;
 
  private:
-  VREyeParametersPtr InitEye(float fov, float offset, uint32_t size);
+  mojom::VREyeParametersPtr InitEye(float fov, float offset, uint32_t size);
+
+  mojom::VRDisplayInfoPtr device_;
+  mojom::VRPosePtr pose_;
 
-  VRDisplayPtr device_;
-  VRPosePtr pose_;
+  unsigned int updated_;

[dcheng, 2016/11/08 08:55:16]

Nit: please name this field more descriptively. number_of_updated_devices_ is
what this means, I think?

Similar comment applies to the accessor.

 
   DISALLOW_COPY_AND_ASSIGN(FakeVRDevice);
 };