mojo VR interface simplified

device/vr/test/fake_vr_device.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 DEVICE_VR_TEST_FAKE_VR_DEVICE_H_
 #define DEVICE_VR_TEST_FAKE_VR_DEVICE_H_
 
 #include "base/macros.h"
 #include "device/vr/vr_device.h"
 #include "device/vr/vr_device_provider.h"
 
 namespace device {
 
 class FakeVRDevice : public VRDevice {
  public:
   explicit FakeVRDevice(VRDeviceProvider* provider);
   ~FakeVRDevice() override;
 
   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; }
+  bool IsDeviceRegistered() {
+    device_registered = !device_registered;

[dcheng, 2016/11/04 05:12:21]

Why does this mutate device_registered?

[shaobo.yan, 2016/11/05 08:11:38]

It means to reset device_registered status when calling this method.
It's an old method which will be updated by following CL.

+    return !device_registered;
+  }
 
-  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);
 
-  VRDisplayPtr device_;
-  VRPosePtr pose_;
+  mojom::VRDisplayInfoPtr device_;
+  mojom::VRPosePtr pose_;
+
+  unsigned updated;

[dcheng, 2016/11/04 05:12:21]

Nit: fields should end with _

[shaobo.yan, 2016/11/05 08:11:38]

thx and done.

+  bool device_registered;
 
   DISALLOW_COPY_AND_ASSIGN(FakeVRDevice);
 };
 
 }  // namespace device
 
 #endif  // DEVICE_VR_TEST_FAKE_VR_DEVICE_H_