mirrors/electron
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions

Revert "fix: better window hierarchy checks"

Browse source 
This reverts commit 8e368a046d.
This commit is contained in:
Samuel Attard 2020-03-23 19:34:52 -07:00
parent 8e368a046d
commit 6b41d86032
No known key found for this signature in database
GPG key ID: FB94249299E904FE
1 changed files with 3 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
lib/browser/guest-window-manager.js
View file

@ -1,7 +1,6 @@
'use strict'; 'use strict';
const electron = require('electron'); const electron = require('electron');
const nodeUrl = require('url');
const { BrowserWindow } = electron; const { BrowserWindow } = electron;
const { isSameOrigin } = process.electronBinding('v8_util'); const { isSameOrigin } = process.electronBinding('v8_util');
const { ipcMainInternal } = require('@electron/internal/browser/ipc-main-internal'); const { ipcMainInternal } = require('@electron/internal/browser/ipc-main-internal');
@ -182,8 +181,9 @@ const isNodeIntegrationEnabled = function (sender) {
// Checks whether |sender| can access the |target|: // Checks whether |sender| can access the |target|:
const canAccessWindow = function (sender, target) { const canAccessWindow = function (sender, target) {
return isScriptableWindow(sender, target) || return isChildWindow(sender, target) ||
(isChildWindow(sender, target) && isNodeIntegrationEnabled(sender)); isScriptableWindow(sender, target) ||
isNodeIntegrationEnabled(sender);
}; };
// Routed window.open messages with raw options // Routed window.open messages with raw options
@ -191,12 +191,6 @@ ipcMainInternal.on('ELECTRON_GUEST_WINDOW_MANAGER_WINDOW_OPEN', (event, url, fra
if (url == null || url === '') url = 'about:blank'; if (url == null || url === '') url = 'about:blank';
if (frameName == null) frameName = ''; if (frameName == null) frameName = '';
if (features == null) features = ''; if (features == null) features = '';
const parsedSourceURL = nodeUrl.parse(event.sender.getURL());
const parsedTargetURL = nodeUrl.parse(url);
if (parsedTargetURL.protocol === 'file:' && parsedSourceURL.protocol !== 'file:') {
event.returnValue = null;
return;
}
const options = {}; const options = {};