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

Unified Diff: device/vr/vr_service.mojom

Issue 2612333002: Allow VRDisplay to specify which frame the layer bounds should be updated at. (Closed)
Patch Set: rebase + address comments Created 3 years, 11 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
« no previous file with comments | « device/vr/vr_display_impl.cc ('k') | third_party/WebKit/Source/modules/vr/VRDisplay.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: device/vr/vr_service.mojom
diff --git a/device/vr/vr_service.mojom b/device/vr/vr_service.mojom
index b7d8515825e0a45e1f521f051a7e276f5d0c41dc..19717b8c5cffa81b52babb983ae5ed7b2221627c 100644
--- a/device/vr/vr_service.mojom
+++ b/device/vr/vr_service.mojom
@@ -1,5 +1,6 @@
// 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.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
module device.mojom;
@@ -23,9 +24,6 @@ struct VRPose {
array<float, 3>? linearVelocity;
array<float, 3>? angularAcceleration;
array<float, 3>? linearAcceleration;
- // The poseIndex is a sequential ID that's incremented on each distinct
- // getPose result, it may wrap around for long sessions.
- uint32 poseIndex;
};
struct VRDisplayCapabilities {
@@ -72,17 +70,19 @@ enum VRDisplayEventReason {
UNMOUNTED = 3
};
-// TODO(shaobo.yan@intel.com) : Add comments to describe these interfaces about how to use and where they live.
+// TODO(shaobo.yan@intel.com) : Add comments to describe these interfaces about
+// how to use and where they live.
interface VRService {
- // TODO(shaobo.yan@intel.com) : Use a factory function which took a VRServiceClient
- // so we would never have a half-initialized VRService.
+ // TODO(shaobo.yan@intel.com) : Use a factory function which took a
+ // VRServiceClient so we would never have a half-initialized VRService.
SetClient(VRServiceClient client) => (uint32 numberOfConnectedDevices);
// Inform the service that the page is listening for vrdisplayactivate events.
SetListeningForActivate(bool listening);
};
interface VRServiceClient {
- OnDisplayConnected(VRDisplay display, VRDisplayClient& request, VRDisplayInfo displayInfo);
+ OnDisplayConnected(VRDisplay display, VRDisplayClient& request,
+ VRDisplayInfo displayInfo);
};
interface VRDisplay {
@@ -91,12 +91,16 @@ interface VRDisplay {
RequestPresent(bool secureOrigin) => (bool success);
ExitPresent();
SubmitFrame(VRPose? pose);
- UpdateLayerBounds(VRLayerBounds leftBounds, VRLayerBounds rightBounds);
+ UpdateLayerBounds(int16 frameId, VRLayerBounds leftBounds,
+ VRLayerBounds rightBounds);
GetVRVSyncProvider(VRVSyncProvider& request);
};
interface VRVSyncProvider {
- GetVSync() => (VRPose? pose, mojo.common.mojom.TimeDelta time);
+ // The frameId maps a VSync to a frame arriving from the compositor. IDs will
+ // be reused after the frame arrives from the compositor. Negative IDs imply
+ // no mapping.
+ GetVSync() => (VRPose? pose, mojo.common.mojom.TimeDelta time, int16 frameId);
};
interface VRDisplayClient {
« no previous file with comments | « device/vr/vr_display_impl.cc ('k') | third_party/WebKit/Source/modules/vr/VRDisplay.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698