Title: [278563] trunk/Source/WebCore
Revision
278563
Author
ifernan...@igalia.com
Date
2021-06-07 10:00:31 -0700 (Mon, 07 Jun 2021)

Log Message

Change WebXRSpace::efectiveOrigin() matrix multiplication order
https://bugs.webkit.org/show_bug.cgi?id=224364
<rdar://problem/76754680>

Reviewed by Sergio Villar Senin.

The current multiplication order is not working correctly when combining local-floor space and getOfffsetReferenceSpace().
The reference space is out of sync with input ray pose after some teleportations:
https://github.com/immersive-web/webxr-samples/blob/main/teleportation.html

Tested using WebXR WPT tests.

* Modules/webxr/WebXRSpace.cpp:
(WebCore::WebXRSpace::effectiveOrigin const):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (278562 => 278563)


--- trunk/Source/WebCore/ChangeLog	2021-06-07 15:37:54 UTC (rev 278562)
+++ trunk/Source/WebCore/ChangeLog	2021-06-07 17:00:31 UTC (rev 278563)
@@ -1,5 +1,22 @@
 2021-06-07  Imanol Fernandez  <ifernan...@igalia.com>
 
+        Change WebXRSpace::efectiveOrigin() matrix multiplication order
+        https://bugs.webkit.org/show_bug.cgi?id=224364
+        <rdar://problem/76754680>
+
+        Reviewed by Sergio Villar Senin.
+
+        The current multiplication order is not working correctly when combining local-floor space and getOfffsetReferenceSpace().
+        The reference space is out of sync with input ray pose after some teleportations:
+        https://github.com/immersive-web/webxr-samples/blob/main/teleportation.html
+
+        Tested using WebXR WPT tests.
+
+        * Modules/webxr/WebXRSpace.cpp:
+        (WebCore::WebXRSpace::effectiveOrigin const):
+
+2021-06-07  Imanol Fernandez  <ifernan...@igalia.com>
+
         Use 3D points to map the WebXRBoundedReferenceSpace bounds transform
         https://bugs.webkit.org/show_bug.cgi?id=224365
         <rdar://problem/76754927>

Modified: trunk/Source/WebCore/Modules/webxr/WebXRSpace.cpp (278562 => 278563)


--- trunk/Source/WebCore/Modules/webxr/WebXRSpace.cpp	2021-06-07 15:37:54 UTC (rev 278562)
+++ trunk/Source/WebCore/Modules/webxr/WebXRSpace.cpp	2021-06-07 17:00:31 UTC (rev 278563)
@@ -50,7 +50,7 @@
 {
     // https://immersive-web.github.io/webxr/#xrspace-effective-origin
     // The effective origin can be obtained by multiplying origin offset and the native origin.
-    return m_originOffset->rawTransform() * nativeOrigin();
+    return nativeOrigin() * m_originOffset->rawTransform();
 }
 
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to