Encode frame number in pixel data for pose sync

device/vr/vr_service.mojom

 // 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.
 
 module device;
 
 // A field of view, given by 4 degrees describing the view from a center point.
 struct VRFieldOfView {
   float upDegrees;
   float downDegrees;
   float leftDegrees;
   float rightDegrees;
 };
 
 // A display's position, orientation, velocity, and acceleration state at the
-// given timestamp.
+// given timestamp. The poseNum is a sequential ID that's incremented on

[mthiesse, 2016/10/04 21:19:17]

Again, I think frameIndex is less confusing. The frame index this pose applies
to.

[klausw, 2016/10/04 23:08:14]

Renamed to poseIndex.

+// each distinct getPose result, it may wrap around for long sessions.
 struct VRPose {
   double timestamp;
   array<float, 4>? orientation;
   array<float, 3>? position;
   array<float, 3>? angularVelocity;
   array<float, 3>? linearVelocity;
   array<float, 3>? angularAcceleration;
   array<float, 3>? linearAcceleration;
+  uint32 poseNum;

[dcheng, 2016/10/04 23:17:53]

Nit: can we move the newly added comment here, so it's next to the thing it
defines?

[klausw, 2016/10/04 23:56:42]

Done.

 };
 
 struct VRDisplayCapabilities {
   bool hasOrientation;
   bool hasPosition;
   bool hasExternalDisplay;
   bool canPresent;
 };
 
 // Information about the optical properties for an eye in a VRDisplay.
@@ skipping 39 matching lines @@
   SubmitFrame(uint32 index, VRPose? pose);
   UpdateLayerBounds(uint32 index, VRLayerBounds leftBounds, VRLayerBounds rightBounds);
 };
 
 interface VRServiceClient {
   OnDisplayChanged(VRDisplay display);
   OnExitPresent(uint32 index);
   OnDisplayConnected(VRDisplay display);
   OnDisplayDisconnected(uint32 index);
 };