Device Service: Move //device/vr to other location(s) in the codebase |
||||||||
Issue descriptionBlocked on adding the ability to mock service connectors in JS for layout tests (see blocking bug). Note that the interface should have a TODO referencing crbug.com/680518 to make clear that the hosted interface won't do anything useful unless the embedder of the Device Service has injected the needed dependencies into the VR impl via C++. As part of this work, the Mojo interface should be moved into //services/device/public/interfaces.
,
May 19 2017
There is a CL in pending: https://codereview.chromium.org/2603553002/
,
May 19 2017
,
Jul 7 2017
,
Sep 28 2017
Updating based on our new understanding that VR will not end up in the Device Service but rather in a distinct service. Thus, servicification of VR is not a blocker for the Device Service. However, we *do* need to eliminate //device/vr as part of eliminating //device. My proposal based on various discussions: - Move the Mojo interfaces into Blink so that both Blink and //chrome can access them - Move all of the C++ code into //chrome (presumably //chrome/browser/vr, although perhaps //device/vr/android should go somewhere else, e.g. //chrome/browser/android/vr_shell). VR folks, does that sound feasible?
,
Sep 28 2017
It'll be interesting to figure out how we want to wire up the gamepads :/
,
Sep 29 2017
re: c#7, I commented on the Gamepad bug raising this issue: https://bugs.chromium.org/p/chromium/issues/detail?id=612330#c43.
,
Nov 7 2017
,
Dec 11 2017
Per input from bajones@, there is a tentative plan to eliminate VR's dependence on Gamepad, but the timeframe is not clear (see blocking bug).
,
Dec 11
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot |
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 Deleted