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: cherry pick 9009d76968b1ec2ed825bc95e47d086ceea07520 from chromium #40681

Merged

Conversation

ryanthemanuel
Copy link
Contributor

@ryanthemanuel ryanthemanuel commented Dec 2, 2023

Description of Change

This PR adds a chromium patch to address this issue: https://bugs.chromium.org/p/chromium/issues/detail?id=1502591

This issue was corrected in chromium version 121.0.6157.0 in CL: chromium-review.googlesource.com/c/chromium/src/+/5033885

I've added this patch in an effort to get this fix sooner and potentially in older versions of electron as well.

Checklist

Release Notes

Notes: Fixed an issue where font requests were incorrectly being sent to dev tools multiple times per resource.

@ryanthemanuel ryanthemanuel requested a review from a team as a code owner December 2, 2023 02:48
@electron-cation electron-cation bot added the new-pr 🌱 PR opened in the last 24 hours label Dec 2, 2023
@deepak1556 deepak1556 added semver/patch backwards-compatible bug fixes no-backport labels Dec 4, 2023
@electron-cation electron-cation bot removed the new-pr 🌱 PR opened in the last 24 hours label Dec 4, 2023
@deepak1556 deepak1556 merged commit 6f14dba into electron:main Dec 4, 2023
18 checks passed
Copy link

release-clerk bot commented Dec 4, 2023

Release Notes Persisted

Fixed an issue where font requests were incorrectly being sent to dev tools multiple times per resource.

@electron-cation electron-cation bot added new-pr 🌱 PR opened in the last 24 hours and removed new-pr 🌱 PR opened in the last 24 hours labels Dec 4, 2023
@ryanthemanuel
Copy link
Contributor Author

ryanthemanuel commented Dec 4, 2023

Thanks for the quick merge! @deepak1556 is there any chance that this can get back ported?

@deepak1556 deepak1556 added target/27-x-y PR should also be added to the "27-x-y" branch. target/28-x-y PR should also be added to the "28-x-y" branch. and removed no-backport labels Dec 4, 2023
@deepak1556
Copy link
Member

/trop run backport

@trop
Copy link
Contributor

trop bot commented Dec 4, 2023

The backport process for this PR has been manually initiated - here we go! :D

@trop
Copy link
Contributor

trop bot commented Dec 4, 2023

I have automatically backported this PR to "27-x-y", please check out #40687

@trop trop bot added in-flight/27-x-y and removed target/27-x-y PR should also be added to the "27-x-y" branch. labels Dec 4, 2023
@trop
Copy link
Contributor

trop bot commented Dec 4, 2023

I have automatically backported this PR to "28-x-y", please check out #40688

@trop trop bot added in-flight/28-x-y merged/28-x-y PR was merged to the "28-x-y" branch. and removed target/28-x-y PR should also be added to the "28-x-y" branch. in-flight/28-x-y labels Dec 4, 2023
ryanthemanuel added a commit to ryanthemanuel/electron that referenced this pull request Dec 6, 2023
See that PR for details.

Notes: Fixed an issue where font requests were incorrectly being sent to dev tools multiple times per resource.
@trop
Copy link
Contributor

trop bot commented Dec 6, 2023

@ryanthemanuel has manually backported this PR to "27-x-y", please check out #40715

ryanthemanuel added a commit to ryanthemanuel/electron that referenced this pull request Dec 7, 2023
See that PR for details.

Notes: Fixed an issue where font requests were incorrectly being sent to dev tools multiple times per resource.
@trop trop bot removed the in-flight/27-x-y label Dec 7, 2023
zcbenz pushed a commit that referenced this pull request Dec 11, 2023
…um (#40715)

Backport of #40681

See that PR for details.

Notes: Fixed an issue where font requests were incorrectly being sent to dev tools multiple times per resource.
@trop trop bot added the merged/27-x-y PR was merged to the "27-x-y" branch. label Dec 11, 2023
MrHuangJser pushed a commit to MrHuangJser/electron that referenced this pull request Dec 11, 2023
…um (electron#40681)

* chore: cherry pick 9009d76968b1ec2ed825bc95e47d086ceea07520 from chromium

* update patch message
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged/27-x-y PR was merged to the "27-x-y" branch. merged/28-x-y PR was merged to the "28-x-y" branch. semver/patch backwards-compatible bug fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants