Bug 219171 - Turn on ENABLE_WEBXR for Cocoa
Summary: Turn on ENABLE_WEBXR for Cocoa
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebXR (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Ada Chan
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2020-11-19 11:24 PST by Ada Chan
Modified: 2020-11-19 20:33 PST (History)
10 users (show)

See Also:


Attachments
Patch (34.39 KB, patch)
2020-11-19 14:54 PST, Ada Chan
no flags Details | Formatted Diff | Diff
Patch (30.14 KB, patch)
2020-11-19 15:40 PST, Ada Chan
no flags Details | Formatted Diff | Diff
Patch (30.13 KB, patch)
2020-11-19 15:45 PST, Ada Chan
no flags Details | Formatted Diff | Diff
Patch (30.10 KB, patch)
2020-11-19 17:03 PST, Ada Chan
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ada Chan 2020-11-19 11:24:15 PST
Turn on ENABLE_WEBXR for Cocoa

rdar://71524773
Comment 1 Ada Chan 2020-11-19 14:54:23 PST
Created attachment 414623 [details]
Patch
Comment 2 Ada Chan 2020-11-19 15:40:34 PST
Created attachment 414627 [details]
Patch
Comment 3 Ada Chan 2020-11-19 15:45:05 PST
Created attachment 414629 [details]
Patch
Comment 4 Tim Horton 2020-11-19 15:53:57 PST
Comment on attachment 414629 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=414629&action=review

> Source/WebCore/Modules/webxr/WebXRRigidTransform.h:41
> +    WTF_MAKE_ISO_ALLOCATED_EXPORT(WebXRRigidTransform, WEBCORE_EXPORT);

Are these for WebKit's use, or just WebCoreTestSupport? If the latter, you can WEBCORE_TESTSUPPORT_EXPORT

> Source/WebCore/platform/xr/openxr/PlatformXROpenXR.cpp:20
> +#if ENABLE(WEBXR) && USE_OPENXR

should be USE(OPENXR), not USE_OPENXR. Ditto in a bunch of places.
Comment 5 Tim Horton 2020-11-19 16:47:06 PST
(In reply to Tim Horton from comment #4)
> Comment on attachment 414629 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=414629&action=review
> 
> > Source/WebCore/Modules/webxr/WebXRRigidTransform.h:41
> > +    WTF_MAKE_ISO_ALLOCATED_EXPORT(WebXRRigidTransform, WEBCORE_EXPORT);
> 
> Are these for WebKit's use, or just WebCoreTestSupport? If the latter, you
> can WEBCORE_TESTSUPPORT_EXPORT

/nevermind/! That is not the macro I thought it was.
Comment 6 Ada Chan 2020-11-19 17:03:26 PST
Created attachment 414642 [details]
Patch
Comment 7 EWS 2020-11-19 20:33:08 PST
Committed r270067: <https://trac.webkit.org/changeset/270067>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 414642 [details].