Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(250)

Unified Diff: device/vr/test/fake_vr_device.cc

Issue 2746233002: Fixes 2D-to-WebVR site transitions when browsing in VR. (Closed)
Patch Set: Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: device/vr/test/fake_vr_device.cc
diff --git a/device/vr/test/fake_vr_device.cc b/device/vr/test/fake_vr_device.cc
index c699905019dff22bee1b1b63e2c6d2f13e57e77e..a94dcbdcb3eebbeb0100822c07ee3c94cf4f1b08 100644
--- a/device/vr/test/fake_vr_device.cc
+++ b/device/vr/test/fake_vr_device.cc
@@ -7,7 +7,7 @@
namespace device {
FakeVRDevice::FakeVRDevice() {
- device_ = mojom::VRDisplayInfo::New();
+ display_info_ = mojom::VRDisplayInfo::New();
InitBasicDevice();
}
@@ -15,17 +15,17 @@ FakeVRDevice::FakeVRDevice() {
FakeVRDevice::~FakeVRDevice() {}
void FakeVRDevice::InitBasicDevice() {
- device_->index = id();
- device_->displayName = "FakeVRDevice";
+ display_info_->index = id();
+ display_info_->displayName = "FakeVRDevice";
- device_->capabilities = mojom::VRDisplayCapabilities::New();
- device_->capabilities->hasOrientation = true;
- device_->capabilities->hasPosition = false;
- device_->capabilities->hasExternalDisplay = false;
- device_->capabilities->canPresent = false;
+ display_info_->capabilities = mojom::VRDisplayCapabilities::New();
+ display_info_->capabilities->hasOrientation = true;
+ display_info_->capabilities->hasPosition = false;
+ display_info_->capabilities->hasExternalDisplay = false;
+ display_info_->capabilities->canPresent = false;
- device_->leftEye = InitEye(45, -0.03f, 1024);
- device_->rightEye = InitEye(45, 0.03f, 1024);
+ display_info_->leftEye = InitEye(45, -0.03f, 1024);
+ display_info_->rightEye = InitEye(45, 0.03f, 1024);
}
mojom::VREyeParametersPtr FakeVRDevice::InitEye(float fov,
@@ -51,13 +51,13 @@ mojom::VREyeParametersPtr FakeVRDevice::InitEye(float fov,
}
void FakeVRDevice::SetVRDevice(const mojom::VRDisplayInfoPtr& device) {
amp 2017/03/13 22:26:05 Any reason not to rename the param here to display
tiborg 2017/03/13 23:00:58 Good call! Changed it.
- device_ = device.Clone();
+ display_info_ = device.Clone();
}
-void FakeVRDevice::GetVRDevice(
- const base::Callback<void(mojom::VRDisplayInfoPtr)>& callback) {
- mojom::VRDisplayInfoPtr display = device_.Clone();
- callback.Run(std::move(display));
+void FakeVRDevice::CreateVRDisplayInfo(
+ const base::Callback<void(mojom::VRDisplayInfoPtr)>& onCreated) {
+ mojom::VRDisplayInfoPtr display = display_info_.Clone();
+ onCreated.Run(std::move(display));
}
void FakeVRDevice::ResetPose() {}
« no previous file with comments | « device/vr/test/fake_vr_device.h ('k') | device/vr/vr_device.h » ('j') | device/vr/vr_device.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698