-
Notifications
You must be signed in to change notification settings - Fork 15.4k
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
chore: bump chromium to 122.0.6194.0 (main) #40750
Conversation
cedeebb
to
ea0df76
Compare
feb71d6
to
85b7696
Compare
Xref: https://chromium-review.googlesource.com/c/chromium/src/+/5079532 minor manual changes to match upstream changes
Xref: https://chromium-review.googlesource.com/c/chromium/src/+/5094777 Xref: https://chromium-review.googlesource.com/c/chromium/src/+/5054299 Update patch to upstream changes in print_view_manager_base.cc. CC @codebytere for 👀
Xref: https://chromium-review.googlesource.com/c/chromium/src/+/5083156 no manual changes; patch applied with fuzz 1
Xref: https://chromium-review.googlesource.com/c/chromium/src/+/5087072 minor manual changes to match upstream changes
Xref: https://chromium-review.googlesource.com/c/chromium/src/+/4960481 no manual changes; patch applied with fuzz 2
…ting.patch Xref: https://chromium-review.googlesource.com/c/v8/v8/+/5076902 Xref: https://chromium-review.googlesource.com/c/v8/v8/+/5096253 The `bailout_value` variable, which allowed us to fail gracefully in these macros, has been removed upstream. This raises the temperature on #40741 ... Since we don't have a bailout value, return a default-constructed instance of whatever the return type is of the function that invokes the macro.
Xref: https://chromium-review.googlesource.com/c/chromium/src/+/5068017 It appears it is not used by anything within Chromium or outside.
Xref: https://chromium-review.googlesource.com/c/chromium/src/+/5066872 The copy construtor is more expensive and has been removed upstream
37ce3f8
to
ce89248
Compare
@@ -1451,6 +1451,7 @@ describe('session module', () => { | |||
w.webContents.executeJavaScript(` | |||
var iframe = document.createElement('iframe'); | |||
iframe.src = '${loadUrl}'; | |||
iframe.allow = 'clipboard-read'; |
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.
https://www.w3.org/TR/permissions-policy-1/ is now enabled for cross origin frames, should be documented under breaking changes.
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.
@codebytere can you add the breaking change doc update to this PR?
The rest of the failing tests related to pending exceptions are a side effect of the following change https://chromium-review.googlesource.com/c/v8/v8/+/5050065, haven't gone through the change yet. |
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 pending breaking change doc update.
@@ -1451,6 +1451,7 @@ describe('session module', () => { | |||
w.webContents.executeJavaScript(` | |||
var iframe = document.createElement('iframe'); | |||
iframe.src = '${loadUrl}'; | |||
iframe.allow = 'clipboard-read'; |
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.
@codebytere can you add the breaking change doc update to this PR?
Release Notes Persisted
|
I have automatically backported this PR to "29-x-y", please check out #40872 |
Updating Chromium to 122.0.6194.0.
See all changes in 121.0.6159.0..122.0.6194.0
Notes: Updated Chromium to 122.0.6194.0.