WebVR: add angular velocity estimate to pose

chrome/browser/android/vr_shell/vr_shell.h

Index: chrome/browser/android/vr_shell/vr_shell.h
diff --git a/chrome/browser/android/vr_shell/vr_shell.h b/chrome/browser/android/vr_shell/vr_shell.h
index 7a3de3000f39c4ee92d3cc28a4516f973d1463f3..43944f10f4a8b22074085643c4f4d22acdf60f7b 100644
--- a/chrome/browser/android/vr_shell/vr_shell.h
+++ b/chrome/browser/android/vr_shell/vr_shell.h
@@ -183,7 +183,9 @@ class VrShell : public device::GvrDelegate,
   void RegisterGamepadDataFetcher(device::GvrGamepadDataFetcher*) override;
 
   // TODO(mthiesse): Find a better place for these functions to live.
-  static device::mojom::VRPosePtr VRPosePtrFromGvrPose(gvr::Mat4f head_mat);
+  static device::mojom::VRPosePtr VRPosePtrFromGvrPose(gvr::Mat4f head_mat,

[cjgrant, 2017/03/24 21:15:40]

Could you:
- Document the function and what each matrix is
- s/mat/matrix/ as per the style guide preference to not abbreviate

[klausw, 2017/03/24 22:13:29]

Done, but only using "matrix" in the header for documentation purposes. For the
.cc file I'd prefer to stick with the pre-existing "mat", especially since
writing out matrix would make some lines excessively long.

+                                                       gvr::Mat4f head_mat_2,
+                                                       int64_t epsilon_nanos);
   static gvr::Sizei GetRecommendedWebVrSize(gvr::GvrApi* gvr_api);
   static device::mojom::VRDisplayInfoPtr CreateVRDisplayInfo(
       gvr::GvrApi* gvr_api,