Closed Bug 1341516 Opened 3 years ago Closed 3 years ago

[webvr] Fix Oculus Touch ID and poseState

Categories

(Core :: Graphics, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla54
Tracking Status
firefox54 --- fixed

People

(Reporter: daoshengmu, Assigned: daoshengmu)

Details

Attachments

(1 file)

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.
Assignee: nobody → dmu
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.
Attachment #8839802 - Flags: review?(kgilbert) → review+
Pushed by dmu@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/004930dad761
Fix Oculus Touch ID and poseState; r=kip
https://hg.mozilla.org/mozilla-central/rev/004930dad761
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
You need to log in before you can comment on or make changes to this bug.