Skip to content

Conversation

@reitowo
Copy link
Member

@reitowo reitowo commented May 30, 2025

This feature provides a way to import a native shared texture handle into Electron, specifically in the form of VideoFrame, which by nature supports several Web rendering systems including WebGPU, WebGL. This enables developers to integrate arbitrary native rendered content with their web applications.

Rendered: https://github.com/reitowo/rfcs/blob/main-shared-texture/text/0017-shared-texture.md

Implementation: electron/electron#47317

@reitowo reitowo force-pushed the main-shared-texture branch from 0c30774 to 05b8645 Compare May 31, 2025 07:36
@samuelmaddock samuelmaddock added the pending-review Waiting for reviewers to give feedback on the PR specifications label Jun 5, 2025
@erickzhao erickzhao added pending-changes Waiting for changes from the PR author based on feedback given and removed pending-review Waiting for reviewers to give feedback on the PR specifications labels Jul 3, 2025
@reitowo
Copy link
Member Author

reitowo commented Aug 9, 2025

I decided to add 2 methods about passing syncToken, then we can manage the lifecycle for user at pure JS.

@reitowo
Copy link
Member Author

reitowo commented Aug 13, 2025

Ready for review! I significantly polished up my API design, providing a more straight forward way to use, and use Electron IPC to automatically manage lifecycles for user.

@reitowo reitowo force-pushed the main-shared-texture branch from 18ce4d7 to 3c670e3 Compare August 13, 2025 18:42
@erickzhao erickzhao added pending-review Waiting for reviewers to give feedback on the PR specifications and removed pending-changes Waiting for changes from the PR author based on feedback given labels Aug 14, 2025
Copy link
Member

@samuelmaddock samuelmaddock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

RFC LGTM

Copy link
Member

@jkleinsc jkleinsc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

RFC LGTM

@reitowo
Copy link
Member Author

reitowo commented Sep 29, 2025

@MarshallOfSound @codebytere 🙇‍♂️🙇‍♂️ Could you give a async "RFC LGTM"? Thanks!

Copy link
Member

@MarshallOfSound MarshallOfSound left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

RFC LGTM

@erickzhao erickzhao added final-comment-period and removed pending-review Waiting for reviewers to give feedback on the PR specifications labels Oct 9, 2025
@erickzhao erickzhao merged commit 3a3c156 into electron:main Oct 23, 2025
4 checks passed
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.

6 participants