|Submitter||Diff||Changes||Open Issues||Last Updated|
|Error loading review requests:|
Bug 1288193 - Ensure that the content process receives the presentation status of a VRDisplay immediately after adding or removing layers
58 bytes, text/x-review-board-request
|Details | Review|
When Entering VR, it seems to take 3-4 seconds before anything shows in the headset and when onVRPresentChange is fired. This causes users to either click "Enter VR" again, or think that VR is not functioning. Not sure if this is a Oculus issue or with something with FF?
Assignee: nobody → kgilbert
Ideally, content should respond to the user immediately on requestPresent to prevent users from clicking multiple times while the promise is resolved; however, this delay does seem rather long and will need some investigation. Even if not the case for currently supported VR hardware, content must assume that requestPresent could take multiple seconds due to powering up hardware or waiting for security prompts to be accepted
Component: General → DOM
Product: Firefox → Core
I have found the source of the bug. Patch is incoming..
Attachment #8787778 - Flags: review?(gwright)
Comment on attachment 8787778 [details] Bug 1288193 - Ensure that the content process receives the presentation status of a VRDisplay immediately after adding or removing layers https://reviewboard.mozilla.org/r/76464/#review75234
Attachment #8787778 - Flags: review?(gwright) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/89b9735661b9 Ensure that the content process receives the presentation status of a VRDisplay immediately after adding or removing layers r=gw280
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox51: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
I re-verified using Fx56.0-build6, Fx 57.0b5 and Fx 58.0a1 (build ID: 20171004100049) on Windows 10 x64 and I can confirm this issue is fixed.
Status: RESOLVED → VERIFIED
status-firefox56: --- → verified
status-firefox57: --- → verified
status-firefox58: --- → verified
You need to log in before you can comment on or make changes to this bug.