Closed Bug 1538140 Opened 8 months ago Closed 7 months ago

[wpt-sync] Sync PR 15954 - Remove XRView.viewMatrix

Categories

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

enhancement

Tracking

(firefox68 fixed)

RESOLVED FIXED
mozilla68
Tracking Status
firefox68 --- fixed

People

(Reporter: wptsync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Jacob DeWitt <jacde@chromium.org> wrote:

Remove XRView.viewMatrix

In tests, use XRView.transform.inverse().matrix instead.

This should also fix the issue where XRView.transform was not properly updating on each frame.

Bug: 939615, 944206
Change-Id: Ic28eb75a22ca0f0d230a7f19952df65ce499b405
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1521803
Reviewed-by: Brandon Jones \<bajones@chromium.org>
Reviewed-by: Mounir Lamouri \<mlamouri@chromium.org>
Commit-Queue: Jacob DeWitt \<jacde@chromium.org>
Cr-Commit-Position: refs/heads/master@{#642984}

Failed to get results from try push
Pushed by james@hoppipolla.co.uk:
https://hg.mozilla.org/integration/mozilla-inbound/rev/0388cc929e90
[wpt PR 15954] - Remove XRView.viewMatrix, a=testonly
Pushed by james@hoppipolla.co.uk:
https://hg.mozilla.org/integration/mozilla-inbound/rev/17c3ee86eacb
[wpt PR 15954] - Remove XRView.viewMatrix, a=testonly
Status: NEW → RESOLVED
Closed: 7 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla68
You need to log in before you can comment on or make changes to this bug.