OpenVR controller can't be enumerated after revisiting a WebVR website
Categories
(Core :: WebVR, defect)
Tracking
()
Tracking | Status | |
---|---|---|
firefox68 | --- | fixed |
People
(Reporter: daoshengmu, Assigned: daoshengmu)
Details
Attachments
(1 file)
STR:
- Open https://webvr.info/samples/XX-vr-controllers.html enter VR. You can see your controllers.
- Open an another tab but this one is not a WebVR page, then close the WebVR tab and wait it for a while or checking if the VR process is removed from ProcessExplorer.
- Go to https://webvr.info/samples/XX-vr-controllers.html enter VR again . You can't see your controllers.
According to the investigation in MozRegression
It causes by https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=cfe23417875b17807be4be97c2d03ace5f2fde0d&tochange=d09d3161572aecefd2e47b574e3a14ea8c6ee1d6. But, it was good for while. I am curious if there is some changes since SteamVR 1.2.10. However, if we choose don't reuse the temporary files that we create for actions and manifest. The problem will be solved.
Assignee | ||
Comment 1•5 years ago
|
||
Fortunately, we only enable dom.vr.openvr.action_input
by default in Nightly. If we turn it off, we will be good.
Assignee | ||
Comment 2•5 years ago
|
||
Assignee | ||
Comment 3•5 years ago
|
||
This issue is caused by calling VR_Shutdown() after VRSession is closed, then calling VR_Init() to launch a new session but doesn't point back the file patch again.
We were checking if the action manifest has been existing. If this file has been existing, we give an early return. But, we should still set the file to SetActionManifestPath() again after VR_Init().
Assignee | ||
Comment 4•5 years ago
|
||
I think our openvr action-based input has been good enough to turn it on by default in FF release. I will flip dom.vr.openvr.action_input
on after Bug 1529105 is resolved.
Assignee | ||
Updated•5 years ago
|
Pushed by dmu@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/b6723fd76838 Pointing back to the same VR input action manifest file when the file has been existing. r=kip
Comment 6•5 years ago
|
||
bugherder |
Description
•