-
Notifications
You must be signed in to change notification settings - Fork 15k
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
fix: suppress worldSafe warning emitted from security checks #25692
Conversation
CI failures are unrelated; merging. |
Release Notes Persisted
|
/trop run backport-to 10-x-y |
The backport process for this PR has been manually initiated - |
/trop run backport-to 11-x-y |
The backport process for this PR has been manually initiated - |
I have automatically backported this PR to "10-x-y", please check out #25827 |
I have automatically backported this PR to "11-x-y", please check out #25828 |
Description of Change
The "hello world" Electron app was emitting a warning about executeJavaScript being called without world-safe turned on, even though the client code wasn't calling executeJavaScript. It turns out the culprit was in the security warnings code. For now, suppress the warning by calling
_executeJavaScript
directly.Checklist
Release Notes
Notes: Suppressed a spurious warning about executeJavaScript being called without worldSafeExecuteJavaScript being enabled in apps that do not call executeJavaScript.