Closed
Bug 1148529
Opened 10 years ago
Closed 10 years ago
[e10s] HMDInfoOculus shouldn't instantiate in the content process
Categories
(Core :: Graphics, defect)
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: jimm, Unassigned)
References
Details
(Whiteboard: [gfx-noted])
Spun off from bug 1147830, we should add some XRE_GeckoProcessType() checks to the initialization of this vr code. It was trying to launch in the child.
I'm pretty sure the fix in bug 1147830 works around this, but we should add an explicit process check in here too.
I'm working on a refactor of this in bug 1148009, and we should do this after that.. but I want to fix VR stuff to work with e10s, because e10s is going to be the standard down the line.
I'm mostly worried about the latency issues with e10s, if we end up reading sensor data in the chrome process and have to send it down to content of our ipc. Milliseconds matter here, but I don't know of any way to get around this for now.
Updated•10 years ago
|
Whiteboard: [gfx-noted]
![]() |
Reporter | |
Updated•10 years ago
|
Flags: needinfo?(gwright)
Comment 2•10 years ago
|
||
Since bug 1149566 landed, this is no longer an issue for e10s as we can't have e10s and VR enabled at the same time. Removing tracking-e10s accordingly.
tracking-e10s:
? → ---
Flags: needinfo?(gwright)
![]() |
Reporter | |
Comment 3•10 years ago
|
||
resolved dupe of bug 1149566? then?
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → WORKSFORME
![]() |
Reporter | |
Comment 4•10 years ago
|
||
oops, well, I guess wfm works too. :)
Comment 5•10 years ago
|
||
Vlad talked of leaving this open to track a more long-term fix, but I think he's gone ahead and filed a new bug for that anyway. Worksforme seems good :)
You need to log in
before you can comment on or make changes to this bug.
Description
•