Skip to content

refactor: replace webFrame.routingId with sync IPC#47717

Merged
samuelmaddock merged 3 commits intomainfrom
feat/sync-routing-id
Aug 2, 2025
Merged

refactor: replace webFrame.routingId with sync IPC#47717
samuelmaddock merged 3 commits intomainfrom
feat/sync-routing-id

Conversation

@samuelmaddock
Copy link
Copy Markdown
Member

@samuelmaddock samuelmaddock commented Jul 10, 2025 •
edited
Loading

Description of Change

Follow up to #47616
Depends on #47850

Replaces webFrame.routingId and webFrame.findFrameByRoutingId() with internal synchronous IPC implementations. This allows us to avoid immediate breaking changes and allows application developers time to upgrade.

Checklist

Release Notes

Notes: none

@electron-cation electron-cation bot added the new-pr 🌱 PR opened recently label Jul 10, 2025
itsananderson
itsananderson reviewed Jul 10, 2025
View reviewed changes
lib/renderer/api/web-frame.ts
): Electron.WebFrame | null {
findFrameByRoutingIdDeprecated();
const frameToken = ipcRendererUtils.invokeSync<string | undefined>(
IPC_MESSAGES.BROWSER_GET_FRAME_TOKEN_SYNC,
Copy link
Copy Markdown
Member

@itsananderson itsananderson Jul 10, 2025

Choose a reason for hiding this comment

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

It hadn't occurred to me that this workaround would require one roundtrip to get the routingId, and then a second roundtrip to convert it right back into a frame token 😅

That said, this still seems like the best way to give apps more time to migrate.

Copy link
Copy Markdown
Member Author

@samuelmaddock samuelmaddock Jul 11, 2025

Choose a reason for hiding this comment

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

It's only one roundtrip so it's not too bad! FindFrameByToken only looks within the current renderer process afaik. If the frame is in another process, it won't be returned from the WebFrame API.

electron/shell/renderer/api/electron_api_web_frame.cc

Lines 823 to 826 in a33b599

content::RenderFrame* render_frame =
content::RenderFrame::FromWebFrame(web_frame);
if (render_frame)
return WebFrameRenderer::Create(isolate, render_frame).ToV8();

@codebytere codebytere force-pushed the roller/chromium/main branch from 3da63d1 to b61f468 Compare July 11, 2025 13:33
@samuelmaddock
Copy link
Copy Markdown
Member Author

samuelmaddock commented Jul 11, 2025

I'll wait for the roll to be merged before fixing up this PR.

@codebytere codebytere force-pushed the roller/chromium/main branch from 4db9fea to d3690a9 Compare July 14, 2025 09:36
Base automatically changed from roller/chromium/main to main July 14, 2025 20:42
@electron-cation electron-cation bot removed the new-pr 🌱 PR opened recently label Jul 17, 2025
@samuelmaddock samuelmaddock force-pushed the feat/sync-routing-id branch 2 times, most recently from dd92508 to 18c2fdd Compare July 21, 2025 19:29
@samuelmaddock samuelmaddock changed the base branch from main to feat/wfm-fromframetoken July 21, 2025 19:30
@samuelmaddock samuelmaddock marked this pull request as ready for review July 21, 2025 22:39
@samuelmaddock samuelmaddock requested a review from a team as a code owner July 21, 2025 22:39
@electron-cation electron-cation bot added the new-pr 🌱 PR opened recently label Jul 21, 2025
@samuelmaddock samuelmaddock mentioned this pull request Jul 22, 2025
Open
10 tasks
@electron-cation electron-cation bot removed the new-pr 🌱 PR opened recently label Jul 22, 2025
@samuelmaddock samuelmaddock mentioned this pull request Jul 31, 2025
Merged
5 tasks
Base automatically changed from feat/wfm-fromframetoken to main July 31, 2025 20:41
@samuelmaddock samuelmaddock force-pushed the feat/sync-routing-id branch from 18c2fdd to 3bd7805 Compare July 31, 2025 22:14
@samuelmaddock samuelmaddock added the target/38-x-y PR should also be added to the "38-x-y" branch. label Aug 1, 2025
@samuelmaddock samuelmaddock merged commit bdaf3b9 into main Aug 2, 2025
99 of 101 checks passed
@samuelmaddock samuelmaddock deleted the feat/sync-routing-id branch August 2, 2025 15:00
@release-clerk
Copy link
Copy Markdown

release-clerk bot commented Aug 2, 2025

No Release Notes

@trop trop bot mentioned this pull request Aug 2, 2025
Merged
@trop
Copy link
Copy Markdown
Contributor

trop bot commented Aug 2, 2025

I have automatically backported this PR to "38-x-y", please check out #47941

@trop trop bot added in-flight/38-x-y merged/38-x-y PR was merged to the "38-x-y" branch. and removed target/38-x-y PR should also be added to the "38-x-y" branch. in-flight/38-x-y labels Aug 2, 2025
kigh-ota pushed a commit to kigh-ota/electron that referenced this pull request Sep 30, 2025
@samuelmaddock @kigh-ota
refactor: replace webFrame.routingId with sync IPC (electron#47717)
ecd0dd7 
* refactor: replace webFrame.routingId with sync IPC

* fix: GetConstructor missing isolate

* fix: missing isolate
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@codebytere codebytere codebytere approved these changes

@MarshallOfSound MarshallOfSound Awaiting requested review from MarshallOfSound

+1 more reviewer

@itsananderson itsananderson itsananderson left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

merged/38-x-y PR was merged to the "38-x-y" branch. semver/none

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@samuelmaddock @itsananderson @codebytere