text: Crash in [@ mozilla::gfx::Open
  VRSession::Setup Contoller Actions]
  Categories
  (Core :: WebVR, defect, critical)
  Tracking
  ()
  People
  (Reporter: marcia, Assigned: daoshengmu)
  Details
  (Keywords: crash, regression)
  Crash Data
  Attachments
  (1 file)
  This bug is for crash report bp-e64c4758-7a08-454c-a9e7-182e20190701.
  Seen while looking at nightly crash data:. 21 crashes/5 installs on 69 nightly. No URLS for the nightly crashes. There are also a few single install crashes that happened in 66 and 67.
  Top 10 frames of crashing thread: 0 xul.dll bool mozilla::gfx::OpenVRSession::SetupContollerActions gfx/vr/service/OpenVRSession.cpp:820 1 xul.dll bool mozilla::gfx::OpenVRSession::Initialize gfx/vr/service/OpenVRSession.cpp:311 2 xul.dll void mozilla::gfx::VRService::ServiceInitialize gfx/vr/service/VRService.cpp:260 3 xul.dll nsresult mozilla::detail::RunnableMethodImpl< xpcom/threads/nsThreadUtils.h:1176 4 xul.dll MessageLoop::DoWork ipc/chromium/src/base/message_loop.cc:523 5 xul.dll base::MessagePumpDefault::Run ipc/chromium/src/base/message_pump_default.cc:35 6 xul.dll MessageLoop::RunHandler ipc/chromium/src/base/message_loop.cc:308 7 xul.dll MessageLoop::Run ipc/chromium/src/base/message_loop.cc:290 8 xul.dll base::Thread::ThreadMain ipc/chromium/src/base/thread.cc:192 9 xul.dll `anonymous namespace'::ThreadFunc ipc/chromium/src/base/platform_thread_win.cc:19
  It looks like the only possibility to make this crash is
  vr::VRInput() is null [1], it is odd and only happened at one install, I will keep an eye on it.
  [1]
  I have no idea so far, we should add logs for helping check the error msg. ()
  I think we should check if
  vr::VRInput() is null and its EVRInputError. Then, ask for
  Shutdown().
  This patch should resolve this issue, but I would like wait for Bug 1564203 be solved to make sure there is no other regression.
  Pushed by dmu@mozilla.com: Checking if OpenVR vrinput() is null when doing initialization. r=kip
  Does this patch need a Beta approval request?
  Comment on attachment 9076625 [details]
  Bug 1562679 - Checking if OpenVR vrinput() is null when doing initialization.
  Beta/Release Uplift Approval Request
  - User impact if declined: This crash of
  OpenVRSession::SetupContollerActionswill continue to happen.
  -): This is a right fix for the OpenVR runtime stability. For some users, the VR input API is not ready yet when the VR system is trying to launch. We would like to give it a early return and ask for retry rather than this crash.
  - String changes made/needed:
  Comment on attachment 9076625 [details]
  Bug 1562679 - Checking if OpenVR vrinput() is null when doing initialization.
  Fixes a WebVR crash. Approved for 69.0b5.