feat: add event.processId to be usable with webFrameMain.fromId()

docs/api/structures/ipc-main-event.md

@@ -1,6 +1,7 @@
 # IpcMainEvent Object extends `Event`
 
 * `frameId` Integer - The ID of the renderer frame that sent this message
+* `processId` Integer - The ID of the process which owns the frame
 * `returnValue` any - Set this to the value to be returned in a synchronous message
 * `sender` WebContents - Returns the `webContents` that sent the message
 * `ports` MessagePortMain[] - A list of MessagePorts that were transferred with this message

docs/api/structures/ipc-main-invoke-event.md

@@ -1,4 +1,5 @@
 # IpcMainInvokeEvent Object extends `Event`
 
 * `frameId` Integer - The ID of the renderer frame that sent this message
+* `processId` Integer - The ID of the process which owns the frame
 * `sender` WebContents - Returns the `webContents` that sent the message

docs/api/web-frame-main.md

@@ -26,7 +26,7 @@ win.webContents.on(
 )
 ```
 
-You can also access frames of existing pages by using the `webFrame` property
+You can also access frames of existing pages by using the `mainFrame` property
 of [`WebContents`](web-contents.md).
 
 ```javascript
@@ -51,6 +51,15 @@ async function main () {
 main()
 ```
 
+You can access the frame, which sent an IPC message using `webFrameMain.fromId(event.processId, event.frameId)`.
+```javascript
+const { ipcMain, webFrameMain } = require('electron')
+
+ipcMain.on('ping', (event) => {
+  console.log(`Message from: ${webFrameMain.fromId(event.processId, event.frameId).url}`)
+})
+```
+
 ## Methods
 
 These methods can be accessed from the `webFrameMain` module:

lib/browser/api/web-contents.ts

@@ -1,5 +1,5 @@
-import { app, ipcMain, session, deprecate, BrowserWindowConstructorOptions } from 'electron/main';
-import type { MenuItem, MenuItemConstructorOptions, LoadURLOptions } from 'electron/main';
+import { app, ipcMain, session, deprecate, webFrameMain } from 'electron/main';
+import type { BrowserWindowConstructorOptions, MenuItem, MenuItemConstructorOptions, LoadURLOptions } from 'electron/main';
 
 import * as url from 'url';
 import * as path from 'path';
@@ -493,6 +493,12 @@ const addReplyInternalToEvent = (event: any) => {
   });
 };
 
+const addFrameToEvent = (event: any) => {
+  Object.defineProperty(event, 'frame', {
+    get: () => webFrameMain.fromId(event.processId, event.frameId)
+  });
+}
+
 const addReturnValueToEvent = (event: any) => {
   Object.defineProperty(event, 'returnValue', {
     set: (value) => event.sendReply(value),
@@ -536,6 +542,7 @@ WebContents.prototype._init = function () {
 
   // Dispatch IPC messages to the ipc module.
   this.on('-ipc-message' as any, function (this: Electron.WebContents, event: any, internal: boolean, channel: string, args: any[]) {
+    addFrameToEvent(event);
     if (internal) {
       addReplyInternalToEvent(event);
       ipcMainInternal.emit(channel, event, ...args);
@@ -547,6 +554,7 @@ WebContents.prototype._init = function () {
   });
 
   this.on('-ipc-invoke' as any, function (event: any, internal: boolean, channel: string, args: any[]) {
+    addFrameToEvent(event);
     event._reply = (result: any) => event.sendReply({ result });
     event._throw = (error: Error) => {
       console.error(`Error occurred in handler for '${channel}':`, error);
@@ -561,6 +569,7 @@ WebContents.prototype._init = function () {
   });
 
   this.on('-ipc-message-sync' as any, function (this: Electron.WebContents, event: any, internal: boolean, channel: string, args: any[]) {
+    addFrameToEvent(event);
     addReturnValueToEvent(event);
     if (internal) {
       addReplyInternalToEvent(event);

shell/common/gin_helper/event_emitter.cc

@@ -5,6 +5,7 @@
 #include "shell/common/gin_helper/event_emitter.h"
 
 #include "content/public/browser/render_frame_host.h"
+#include "content/public/browser/render_process_host.h"
 #include "shell/browser/api/event.h"
 #include "shell/common/gin_helper/dictionary.h"
 #include "shell/common/gin_helper/object_template_builder.h"
@@ -67,8 +68,10 @@ v8::Local<v8::Object> CreateNativeEvent(
   Dictionary dict(isolate, event);
   dict.Set("sender", sender);
   // Should always set frameId even when callback is null.
-  if (frame)
+  if (frame) {
+    dict.Set("processId", frame->GetProcess()->GetID());
     dict.Set("frameId", frame->GetRoutingID());
+  }
   return event;
 }
 

spec-main/api-subframe-spec.ts

@@ -35,6 +35,7 @@ describe('renderer nodeIntegrationInSubFrames', () => {
         expect(event1[0].frameId).to.not.equal(event2[0].frameId);
         expect(event1[0].frameId).to.equal(event1[2]);
         expect(event2[0].frameId).to.equal(event2[2]);
+        expect(event2[0].frame.name).to.equal('frame');
       });
 
       it('should load preload scripts in nested iframes', async () => {
@@ -47,6 +48,8 @@ describe('renderer nodeIntegrationInSubFrames', () => {
         expect(event1[0].frameId).to.equal(event1[2]);
         expect(event2[0].frameId).to.equal(event2[2]);
         expect(event3[0].frameId).to.equal(event3[2]);
+        expect(event2[0].frame.name).to.equal('frame-with-frame');
+        expect(event3[0].frame.name).to.equal('frame');
       });
 
       it('should correctly reply to the main frame with using event.reply', async () => {

spec-main/fixtures/sub-frames/frame-container.html

@@ -8,6 +8,6 @@
 </head>
 <body>
   This is the root page
-  <iframe src="./frame.html"></iframe>
+  <iframe src="./frame.html" name="frame"></iframe>
 </body>
 </html>

spec-main/fixtures/sub-frames/frame-with-frame-container.html

@@ -8,6 +8,6 @@
 </head>
 <body>
   This is the root page
-  <iframe src="./frame-with-frame.html"></iframe>
+  <iframe src="./frame-with-frame.html" name="frame-with-frame"></iframe>
 </body>
 </html>