-
Notifications
You must be signed in to change notification settings - Fork 15.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
test: re-enable shared worker webview test #34338
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, very likely addressed with PlzSharedWorker refactor in upstream
const webview = new WebView(); | ||
webview.addEventListener('console-message', (e) => { | ||
console.log(e); | ||
}); | ||
const eventPromise = waitForEvent(webview, 'ipc-message'); | ||
webview.src = `file://${fixtures}/pages/shared_worker.html`; | ||
webview.setAttribute('webpreferences', 'nodeIntegration, nodeIntegrationInWorker'); | ||
webview.setAttribute('webpreferences', 'nodeIntegration, nodeIntegrationInWorker, contextIsolation=no'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good opportunity to refactor the test to work with context isolation https://github.com/electron/electron/blob/main/spec/fixtures/pages/shared_worker.html :)
97f0a73
to
ae401ce
Compare
No Release Notes |
Description of Change
The crash no longer occurs locally and it passes as soon as
contextIsolation=no
is set as an attribute.Checklist
npm test
passesRelease Notes
Notes: none.