WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
224365
Use 3D points to map the WebXRBoundedReferenceSpace bounds transform
https://bugs.webkit.org/show_bug.cgi?id=224365
Summary
Use 3D points to map the WebXRBoundedReferenceSpace bounds transform
Imanol Fernandez
Reported
2021-04-09 04:41:59 PDT
WebXRBoundedReferenceSpace is currently using Point2D mapping to transfom the bounds geometry. This is not correct because the originOffset is a 3D transform and it expects the bounds depth coordinates to be on the z axis instead of the y axis. There is a WPT test to reproduce this but we can't enable it yet until we fetch it from upstream (
https://github.com/web-platform-tests/wpt/pull/28024
), currently blocked by WebGL 2 support in WPE.
Attachments
Patch
(2.36 KB, patch)
2021-04-09 04:51 PDT
,
Imanol Fernandez
no flags
Details
Formatted Diff
Diff
Patch
(5.69 KB, patch)
2021-06-07 06:14 PDT
,
Imanol Fernandez
no flags
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Imanol Fernandez
Comment 1
2021-04-09 04:51:20 PDT
Created
attachment 425610
[details]
Patch
Sergio Villar Senin
Comment 2
2021-04-09 07:56:41 PDT
Comment on
attachment 425610
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=425610&action=review
> Source/WebCore/ChangeLog:13 > + currently blocked by WebGL 2 support in WPE.
WebGL2 shouldn't be a blocker for the testing of this kind of stuff. Why don't we just add a new small test to WPT to check this?
Radar WebKit Bug Importer
Comment 3
2021-04-16 04:42:13 PDT
<
rdar://problem/76754927
>
Imanol Fernandez
Comment 4
2021-06-07 06:14:57 PDT
Created
attachment 430740
[details]
Patch Rebase onto main
EWS
Comment 5
2021-06-07 08:38:01 PDT
Committed
r278562
(
238560@main
): <
https://commits.webkit.org/238560@main
> All reviewed patches have been landed. Closing bug and clearing flags on
attachment 430740
[details]
.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug