Closed Bug 1773593 Opened 2 years ago Closed 2 years ago

[wpt-sync] Sync PR 34370 - Remove VRDisplayInfo and pass initial views to XRSessionDeviceConfig

Categories

(Testing :: web-platform-tests, task, P4)

task

Tracking

(firefox104 fixed)

RESOLVED FIXED
104 Branch
Tracking Status
firefox104 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 34370 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/34370
Details from upstream follow.

Patrick To <patrto@microsoft.com> wrote:

Remove VRDisplayInfo and pass initial views to XRSessionDeviceConfig

VRDisplayInfo currently only has one field - the initial views when the
session starts. This field is now moved into XRSessionDeviceConfig and
VRDisplayInfo is removed completely. These initial views are only used
before any frames are requested. Once frames are requested, updated
views are sent with each XRFrame.

The initial views are still needed because various places assume that
views exist before frames are requested, for example when initializing
a XRWebGLLayer or if the browser renderer needs to take over due to
frames not being requested by the web page. This can be refactored in
a future change if appropriate.

Bug: 862803
Change-Id: Ibba16ff71c0b8da0a39e5a55fcfcc397246ea36e
Reviewed-on: https://chromium-review.googlesource.com/3698752
WPT-Export-Revision: 19bdfcf89c9f30e89ebc7b315d4ee8c95ac9162b

Whiteboard: [wptsync downstream] → [wptsync downstream error]
Whiteboard: [wptsync downstream error] → [wptsync downstream]
Test result changes from PR not available.
Test result changes from PR not available.
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/bfe1da3f1096 [wpt PR 34370] - Remove VRDisplayInfo and pass initial views to XRSessionDeviceConfig, a=testonly
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 104 Branch
You need to log in before you can comment on or make changes to this bug.