Skip to content
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.

Sign up for GitHub

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

Jump to bottom

docs: explain the null webContents case in permission checks #30686

Merged
merged 2 commits into from Aug 26, 2021
Merged

docs: explain the null webContents case in permission checks #30686

merged 2 commits into from Aug 26, 2021

Conversation

trop[bot]
Copy link
Contributor

@trop trop bot commented Aug 25, 2021

Backport of #30645

See that PR for details.

Notes: none

@electron-cation electron-cation bot added the new-pr 🌱 PR opened in the last 24 hours label Aug 25, 2021
@trop trop bot requested a review from MarshallOfSound August 25, 2021 00:42
@trop trop bot mentioned this pull request Aug 25, 2021
Merged
@trop trop bot added 13-x-y backport This is a backport PR semver/none labels Aug 25, 2021
@electron-cation electron-cation bot removed the new-pr 🌱 PR opened in the last 24 hours label Aug 25, 2021
@zcbenz zcbenz merged commit 2917eb1 into 13-x-y Aug 26, 2021
@zcbenz zcbenz deleted the trop/13-x-y-bp-docs-explain-the-null-webcontents-case-in-permission-checks-1629852133150 branch August 26, 2021 06:47
@release-clerk
Copy link

release-clerk bot commented Aug 26, 2021

No Release Notes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zcbenz zcbenz zcbenz approved these changes

@MarshallOfSound MarshallOfSound Awaiting requested review from MarshallOfSound

Assignees
No one assigned
Labels
13-x-y backport This is a backport PR semver/none
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@zcbenz @MarshallOfSound