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.

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: override content::ContentMainDelegate::CreateContentClient() #35964

Conversation

trop[bot]
Copy link
Contributor

@trop trop bot commented Oct 10, 2022

Backport of #35932

See that PR for details.

Notes: Fixed a potential memory leak.

@trop trop bot requested a review from codebytere October 10, 2022 14:49
@trop trop bot added 22-x-y backport This is a backport PR semver/patch backwards-compatible bug fixes labels Oct 10, 2022
trop bot and others added 2 commits October 11, 2022 15:54
Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>
Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>
@deepak1556 deepak1556 force-pushed the trop/22-x-y-bp-fix-override-content-contentmaindelegate-createcontentclient--1665413344737 branch from a0e497e to 7477da4 Compare October 11, 2022 06:54
@MarshallOfSound MarshallOfSound merged commit 07d4fc3 into 22-x-y Oct 11, 2022
@MarshallOfSound MarshallOfSound deleted the trop/22-x-y-bp-fix-override-content-contentmaindelegate-createcontentclient--1665413344737 branch October 11, 2022 09:37
@release-clerk
Copy link

release-clerk bot commented Oct 11, 2022

Release Notes Persisted

Fixed a potential memory leak.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
22-x-y backport This is a backport PR semver/patch backwards-compatible bug fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants