1341516 - [webvr] Fix Oculus Touch ID and poseState

Status: RESOLVED FIXED

(Core :: Graphics, defect)

Description

[Daosheng Mu[:daoshengmu]]

As our investigation, our Touch ID should be `"Oculus Touch (Right)"` instead of `"Oculus Touch (right)"`. Moreover, the state of HandPoses is ordered by ovrControllerType_LTouch and ovrControllerType_RTouch. Therefore, we can't get its state by the index of mOculusController array.

Comment 1

[Daosheng Mu[:daoshengmu]]

Attachment: Bug 1341516 - Fix Oculus Touch ID and poseState;

Review commit: https://reviewboard.mozilla.org/r/114356/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/114356/

Comment 2

[:kip (Kearwood Gilbert)]

Comment on attachment 8839802 [details]
Bug 1341516 - Fix Oculus Touch ID and poseState;

https://reviewboard.mozilla.org/r/114356/#review115814

Thanks for the quick fix! Looks good to me.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/004930dad761
Fix Oculus Touch ID and poseState; r=kip

Comment 4

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/004930dad761