Commit 08245ed3 authored by Brandon Jones's avatar Brandon Jones Committed by Commit Bot

Workaround for WMR UserPresence bug

Provides a workaround for an issue with some versions of Windows Mixed
Reality where the HolographicSpaceUserPresence will only ever report
Absent or PresentPassive, and never PresentActive. We would ideally like
to map PresentPassive to the WebXR state "visible-blurred", as their
meaning is very close, but until this bug is fixed we'll need to force
it to map to "visible" so that content runs correctly.

Bug: 1016907
Change-Id: Icbec972fd6b62d1892609f8cf84ddf9f8190bc44
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1873157Reviewed-by: default avatarAlexander Cooper <alcooper@chromium.org>
Commit-Queue: Brandon Jones <bajones@chromium.org>
Cr-Commit-Position: refs/heads/master@{#708372}
parent 006b8580
...@@ -427,7 +427,11 @@ void MixedRealityRenderLoop::UpdateVisibilityState() { ...@@ -427,7 +427,11 @@ void MixedRealityRenderLoop::UpdateVisibilityState() {
// and the headset is being worn, but a modal dialog is capturing input. // and the headset is being worn, but a modal dialog is capturing input.
case HolographicSpaceUserPresence:: case HolographicSpaceUserPresence::
HolographicSpaceUserPresence_PresentPassive: HolographicSpaceUserPresence_PresentPassive:
SetVisibilityState(device::mojom::XRVisibilityState::VISIBLE_BLURRED); // TODO(1016907): Should report VISIBLE_BLURRED, but changed to VISIBLE to
// work around an issue in some versions of Windows Mixed Reality which
// only report PresentPassive and never PresentActive. Should be reverted
// after the Windows fix has been widely released.
SetVisibilityState(device::mojom::XRVisibilityState::VISIBLE);
return; return;
// Indicates that the browsers immersive content is not visible in the // Indicates that the browsers immersive content is not visible in the
// headset or the user is not wearing the headset. // headset or the user is not wearing the headset.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment