1644506 - [wpt-sync] Sync PR 24065 - WebXR - anchors - fix anchor creation from hit test

Status: RESOLVED FIXED

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

Description

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

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

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

Piotr Bialecki <[e-mail]> wrote:

WebXR - anchors - fix anchor creation from hit test

Anchor creation based off of hit test result was incorrectly using
unstable native origin (planes' origins may change diametrically from
frame to frame) - this CL fixes the issue. Additionally, hit test logic
did not correctly treat hit test results as computed based off of plane
intersection - the issue did not cause any problems with hit test API,
but is problematic for anchor creation API.

Changes:

• plane-attached anchor creation now accepts native origin information
  and pose relative to passed in native origin - this is done to ensure
  that anchor creation uses a frame of reference that is stable w.r.t.
  the environment (plane pose is not a correct choice as it changes when
  plane extents are changing)
• fixed an issue in ArCoreImpl::RequestHitTest() - it would not report
  results as computed based off of a plane even if they were
• mojom's hit test result now uses pose instead of matrix - pose is
  directly provided by ARCore SDK and has smaller cognitive load as it
  does not require considering column-major vs. row-major memory layouts
• IDL - anchor creation based on hit test result and on plane no longer
  accepts initial pose since the spec draft has changed - initial pose
  was considered confusing and unnecessary since it did not extend the
  capabilities of the API

All other changes (parameter renames, comment updates, fixing call
sites to account for parameter count changes, etc.) are related to
the above major changes.

Change-Id: I5acb872c5a9d9003f1f85173edd82e6e6fa002bb
Reviewed-on: https://chromium-review.googlesource.com/2236592
WPT-Export-Revision: 15e364df9fa47158d7a3b6f9734fd2ab19386f0d

Comment 1

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=8bf4c2433c0484b44ba4951c550bbd98c0801ae2

Comment 2

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Test result changes from PR not available.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9390f0534629
[wpt PR 24065] - WebXR - anchors - fix anchor creation from hit test, a=testonly

Comment 4

[Cristian Brindusan [:cbrindusan]]

https://hg.mozilla.org/mozilla-central/rev/9390f0534629