Closed Bug 1341516 Opened 3 years ago Closed 3 years ago

[webvr] Fix Oculus Touch ID and poseState

Categories

(Core :: Graphics, defect)

Product:
Core
Component:
Graphics
Type:
defect
Priority:
Not set

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla54
Tracking Flags:
Tracking Status
firefox54 --- fixed

People

(Reporter: daoshengmu, Assigned: daoshengmu)

Details

Attachments

(1 file)

Bug 1341516 - Fix Oculus Touch ID and poseState;
59 bytes, text/x-review-board-request
kip
: review+
Details 
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
status-firefox54: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
You need to log in before you can comment on or make changes to this bug.