Skip to content

webxr: Remove the ipc feature#41736

Merged
mrobinson merged 1 commit intoservo:mainfrom
Narfinger:webxr-cleanup
Jan 7, 2026
Merged

webxr: Remove the ipc feature#41736
mrobinson merged 1 commit intoservo:mainfrom
Narfinger:webxr-cleanup

Conversation

@Narfinger
Copy link
Copy Markdown
Contributor

@Narfinger Narfinger commented Jan 7, 2026

webxr/webxr-api have the feature flag "ipc". This is required by important crates such as script and canvas. The only distinction was made in servo which does not use the IPC for android/ohos but because servo depends on script, this was essentially useless.

This PR removes the feature flag and cleans up the code related to it.

Testing: Compilation is the test.

@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Jan 7, 2026
@Narfinger Narfinger changed the title WebXR cleanup WebXR feature flag cleanup Jan 7, 2026
Signed-off-by: Narfinger <[email protected]>
@mrobinson mrobinson changed the title WebXR feature flag cleanup webxr: Remove the ipc feature Jan 7, 2026
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Jan 7, 2026
@mrobinson
Copy link
Copy Markdown
Member

@Narfinger Thanks! I've been looking forward to this for a while. A good followup to this change would be to remove the WebXrSender, etc alias.

@mrobinson mrobinson added this pull request to the merge queue Jan 7, 2026
@servo-highfive servo-highfive added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Jan 7, 2026
Merged via the queue into servo:main with commit 8ec23ed Jan 7, 2026
32 checks passed
@servo-highfive servo-highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Jan 7, 2026
@Narfinger Narfinger deleted the webxr-cleanup branch January 7, 2026 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants