Skip to content
This repository has been archived by the owner on Jul 10, 2023. It is now read-only.

Added future_frame_data, to avoid blocking the VR render thread #62

Merged
merged 2 commits into from
Feb 26, 2019

Conversation

asajeffrey
Copy link
Member

At the moment, the VR render thread blocks on the VR device, waiting for the next frame. This PR makes it possible for the VR display to return a future, so the JS thread end up blocking rather than the VR render thread.

@asajeffrey
Copy link
Member Author

Fixes #60

@asajeffrey
Copy link
Member Author

servo/servo#22938 is the matching servo PR.

@paulrouget
Copy link
Contributor

Looks good to me.

@asajeffrey
Copy link
Member Author

@bors-servo r=paulrouget

1 similar comment
@Manishearth
Copy link
Member

@bors-servo r=paulrouget

@Manishearth Manishearth reopened this Feb 26, 2019
@Manishearth
Copy link
Member

@bors-servo r=paulrouget

@bors-servo
Copy link
Contributor

📌 Commit b81bb58 has been approved by paulrouget

bors-servo pushed a commit that referenced this pull request Feb 26, 2019
Added future_frame_data, to avoid blocking the VR render thread

At the moment, the VR render thread blocks on the VR device, waiting for the next frame. This PR makes it possible for the VR display to return a future, so the JS thread end up blocking rather than the VR render thread.
@bors-servo
Copy link
Contributor

⌛ Testing commit b81bb58 with merge 6a4e30c...

@bors-servo
Copy link
Contributor

☀️ Test successful - checks-travis
Approved by: paulrouget
Pushing 6a4e30c to master...

@bors-servo bors-servo merged commit b81bb58 into servo:master Feb 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants