Skip to content

Desktop: Cleaner CEF bidirectional message implementation#2950

Merged
TrueDoctor merged 6 commits intomasterfrom
cef-bidirectional-messages-js-and-main-process
Jul 28, 2025
Merged

Desktop: Cleaner CEF bidirectional message implementation#2950
TrueDoctor merged 6 commits intomasterfrom
cef-bidirectional-messages-js-and-main-process

Conversation

@timon-schelling
Copy link
Copy Markdown
Member

This PR refactors and improves the existing mechanisms from the native-editor branch without altering any current behavior. It introduces a thin abstraction layer for passing raw byte‑array buffers between the renderer (JavaScript) and the main/browser process, to be used in future PRs.

@github-actions github-actions Bot temporarily deployed to graphite-dev (Preview) July 28, 2025 09:42 Inactive
@Keavon Keavon changed the title Desktop: Cleaner cef bidirectional message implementation Desktop: Cleaner CEF bidirectional message implementation Jul 28, 2025
@timon-schelling timon-schelling force-pushed the cef-bidirectional-messages-js-and-main-process branch from 1d898f5 to 3ca0a4f Compare July 28, 2025 10:02
@github-actions github-actions Bot temporarily deployed to graphite-dev (Preview) July 28, 2025 10:33 Inactive
@timon-schelling timon-schelling force-pushed the cef-bidirectional-messages-js-and-main-process branch from 64aba8e to 3661d98 Compare July 28, 2025 10:43
@TrueDoctor TrueDoctor force-pushed the cef-bidirectional-messages-js-and-main-process branch from 3661d98 to e44b46a Compare July 28, 2025 12:45
@TrueDoctor TrueDoctor enabled auto-merge (squash) July 28, 2025 12:45
@timon-schelling
Copy link
Copy Markdown
Member Author

Thanks for rebaseing :)

@github-actions github-actions Bot temporarily deployed to graphite-dev (Preview) July 28, 2025 12:57 Inactive
@TrueDoctor TrueDoctor merged commit 83d39fb into master Jul 28, 2025
4 checks passed
@TrueDoctor TrueDoctor deleted the cef-bidirectional-messages-js-and-main-process branch July 28, 2025 13:04
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.

2 participants