mojo VR interface simplified

device/vr/android/gvr/gvr_device.cc

 // 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 "device/vr/android/gvr/gvr_device.h"
 
 #include <math.h>
 #include <algorithm>
 
 #include "base/time/time.h"
@@ skipping 11 matching lines @@
 
 static const uint64_t kPredictionTimeWithoutVsyncNanos = 50000000;
 
 }  // namespace
 
 GvrDevice::GvrDevice(GvrDeviceProvider* provider, GvrDelegate* delegate)
     : VRDevice(provider), delegate_(delegate), gvr_provider_(provider) {}
 
 GvrDevice::~GvrDevice() {}
 
-VRDisplayPtr GvrDevice::GetVRDevice() {
+mojom::VRDisplayPtr GvrDevice::GetVRDevice() {
   TRACE_EVENT0("input", "GvrDevice::GetVRDevice");
 
-  VRDisplayPtr device = VRDisplay::New();
+  mojom::VRDisplayPtr device = mojom::VRDisplay::New();
 
-  device->index = id();
-
-  device->capabilities = VRDisplayCapabilities::New();
+  device->capabilities = mojom::VRDisplayCapabilities::New();
   device->capabilities->hasOrientation = true;
   device->capabilities->hasPosition = false;
   device->capabilities->hasExternalDisplay = false;
   device->capabilities->canPresent = true;
 
-  device->leftEye = VREyeParameters::New();
-  device->rightEye = VREyeParameters::New();
-  VREyeParametersPtr& left_eye = device->leftEye;
-  VREyeParametersPtr& right_eye = device->rightEye;
+  device->leftEye = mojom::VREyeParameters::New();
+  device->rightEye = mojom::VREyeParameters::New();
+  mojom::VREyeParametersPtr& left_eye = device->leftEye;
+  mojom::VREyeParametersPtr& right_eye = device->rightEye;
 
-  left_eye->fieldOfView = VRFieldOfView::New();
-  right_eye->fieldOfView = VRFieldOfView::New();
+  left_eye->fieldOfView = mojom::VRFieldOfView::New();
+  right_eye->fieldOfView = mojom::VRFieldOfView::New();
 
   left_eye->offset = mojo::Array<float>::New(3);
   right_eye->offset = mojo::Array<float>::New(3);
 
   // TODO(bajones): GVR has a bug that causes it to return bad render target
   // sizes when the phone is in portait mode. Send arbitrary,
   // not-horrifically-wrong values instead.
   //  gvr::Sizei render_target_size = gvr_api->GetRecommendedRenderTargetSize();
   left_eye->renderWidth = 1024;   // render_target_size.width / 2;
   left_eye->renderHeight = 1024;  // render_target_size.height;
@@ skipping 58 matching lines @@
   left_eye->offset[2] = -left_eye_mat.m[2][3];
 
   gvr::Mat4f right_eye_mat = gvr_api->GetEyeFromHeadMatrix(GVR_RIGHT_EYE);
   right_eye->offset[0] = -right_eye_mat.m[0][3];
   right_eye->offset[1] = -right_eye_mat.m[1][3];
   right_eye->offset[2] = -right_eye_mat.m[2][3];
 
   return device;
 }
 
-VRPosePtr GvrDevice::GetPose() {
+mojom::VRPosePtr GvrDevice::GetPose(VRServiceImpl* service) {
   TRACE_EVENT0("input", "GvrDevice::GetSensorState");
 
-  VRPosePtr pose = VRPose::New();
+  if (CheckAccessAllowed(service))
+    return nullptr;
+
+  mojom::VRPosePtr pose = mojom::VRPose::New();
 
   pose->timestamp = base::Time::Now().ToJsTime();
 
   // Increment pose frame counter always, even if it's a faked pose.
   pose->poseIndex = ++pose_index_;
 
   pose->orientation = mojo::Array<float>::New(4);
 
   gvr::GvrApi* gvr_api = GetGvrApi();
   if (!gvr_api) {
@@ skipping 35 matching lines @@
     pose->position[2] = decomposed_transform.translate[2];
   }
 
   // Save the underlying GVR pose for use by rendering. It can't use a
   // VRPosePtr since that's a different data type.
   delegate_->SetGvrPoseForWebVr(head_mat, pose_index_);
 
   return pose;
 }
 
-void GvrDevice::ResetPose() {
+void GvrDevice::ResetPose(VRServiceImpl* service) {
+  if (CheckAccessAllowed(service))
+    return;
+
   gvr::GvrApi* gvr_api = GetGvrApi();
   if (gvr_api)
     gvr_api->ResetTracking();
 }
 
-bool GvrDevice::RequestPresent(bool secure_origin) {
+bool GvrDevice::RequestPresent(VRServiceImpl* service, bool secure_origin) {
+  if (presenting_service_ != nullptr)
+    if (presenting_service_ != service)
+      return false;

[bajones, 2016/10/25 22:21:37]

Any reason why CheckAccessAllowed isn't used here?

+
+  // One service could present on several devices at the same time
+  // and different service could present on different devices the same time
+  if (presenting_service_ == nullptr)
+    presenting_service_ = service;
+
   secure_origin_ = secure_origin;
   if (delegate_)
     delegate_->SetWebVRSecureOrigin(secure_origin_);
-  return gvr_provider_->RequestPresent();
+
+  if (!gvr_provider_->RequestPresent()) {

[bajones, 2016/10/25 22:21:37]

This change is unnecessary.

+    return false;
+  }
+  return true;
 }
 
-void GvrDevice::ExitPresent() {
+void GvrDevice::ExitPresent(VRServiceImpl* service) {
+  if (IsPresentingService(service))
+    presenting_service_ = nullptr;
+
   gvr_provider_->ExitPresent();
+  OnExitPresent(service);
 }
 
-void GvrDevice::SubmitFrame(VRPosePtr pose) {
-  if (delegate_)
-    delegate_->SubmitWebVRFrame();
+void GvrDevice::SubmitFrame(VRServiceImpl* service, mojom::VRPosePtr pose) {
+  if (IsPresentingService(service)) {
+    if (delegate_)
+      delegate_->SubmitWebVRFrame();
+  }
 }
 
-void GvrDevice::UpdateLayerBounds(VRLayerBoundsPtr leftBounds,
-                                  VRLayerBoundsPtr rightBounds) {
-  if (!delegate_)
-    return;
+void GvrDevice::UpdateLayerBounds(VRServiceImpl* service,
+                                  mojom::VRLayerBoundsPtr leftBounds,
+                                  mojom::VRLayerBoundsPtr rightBounds) {
+  if (CheckAccessAllowed(service)) {
+    if (!delegate_)
+      return;
 
-  delegate_->UpdateWebVRTextureBounds(0,  // Left eye
-                                      leftBounds->left, leftBounds->top,
-                                      leftBounds->width, leftBounds->height);
-  delegate_->UpdateWebVRTextureBounds(1,  // Right eye
-                                      rightBounds->left, rightBounds->top,
-                                      rightBounds->width, rightBounds->height);
+    delegate_->UpdateWebVRTextureBounds(0,  // Left eye
+                                        leftBounds->left, leftBounds->top,
+                                        leftBounds->width, leftBounds->height);
+    delegate_->UpdateWebVRTextureBounds(1,  // Right eye
+                                        rightBounds->left, rightBounds->top,
+                                        rightBounds->width,
+                                        rightBounds->height);
+  }
 }
 
 void GvrDevice::SetDelegate(GvrDelegate* delegate) {
   delegate_ = delegate;
 
   // Notify the clients that this device has changed
   if (delegate_) {
     delegate_->SetWebVRSecureOrigin(secure_origin_);
-    VRDeviceManager::GetInstance()->OnDeviceChanged(GetVRDevice());
+    this->OnDisplayChanged();

[bajones, 2016/10/25 22:21:38]

just OnDisplayChanged(), please.

[shaobo.yan, 2016/10/26 01:19:22]

Sorry about these three fault.

   }
 }
 
 gvr::GvrApi* GvrDevice::GetGvrApi() {
   if (!delegate_)
     return nullptr;
 
   return delegate_->gvr_api();
 }
 
+void GvrDevice::OnDisplayChanged() {

[bajones, 2016/10/25 22:21:38]

There's nothing GvrDevice specific about any of these functions. Can we move the
implementation to vr_device.cc so they can be shared by all VRDevices?

[shaobo.yan, 2016/10/26 01:19:22]

Sure and if any kind of devices need different behaviour, they could override
it.

+  mojom::VRDisplayPtr vr_device_info = GetVRDevice();
+  if (vr_device_info.is_null())
+    return;
+
+  for (const auto& client : display_clients_)
+    client.second->OnDisplayChanged(vr_device_info.Clone());
+}
+
+void GvrDevice::OnExitPresent(VRServiceImpl* service) {
+  DisplayClientMap::iterator it = display_clients_.find(service);
+  if (it != display_clients_.end())
+    it->second->OnExitPresent();
+}
+
+void GvrDevice::OnDisplayConnected() {
+  mojom::VRDisplayPtr vr_device_info = GetVRDevice();
+  if (vr_device_info.is_null())
+    return;
+
+  for (const auto& client : display_clients_)
+    client.second->OnDisplayConnected(vr_device_info.Clone());
+}
+
+void GvrDevice::OnDisplayDisconnected() {
+  for (const auto& client : display_clients_)
+    client.second->OnDisplayDisconnected();
+}
+
 }  // namespace device