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: BrowserView background color in webContents #33435
Conversation
122186e
to
fe08a47
Compare
176f6f7
to
7f266ff
Compare
Release Notes Persisted
|
I was unable to backport this PR to "16-x-y" cleanly; |
I was unable to backport this PR to "17-x-y" cleanly; |
I have automatically backported this PR to "18-x-y", please check out #33478 |
@codebytere has manually backported this PR to "17-x-y", please check out #33545 |
@codebytere has manually backported this PR to "16-x-y", please check out #33546 |
* chore: fix BrowserView background color in webContents * disable screen capture test on linux * spec: fix platform failure condition
@codebytere this fix regressed #31722 |
* chore: fix BrowserView background color in webContents * disable screen capture test on linux * spec: fix platform failure condition
Description of Change
Closes #33374.
Fixes an issue where backgroundColor would not be applied correctly to a BrowserView on Windows. This was happening because the color was being set on the
RenderWidgetHostView
in webContents, but the color was missing fromwebPreferences
and so was transparent every time. We need to ensure the color is set properly on the window as well as the contents every timesetBackgroundColor
is called, as is done onBrowserWindow
.Checklist
npm test
passesRelease Notes
Notes: Fixed an issue with background colors being improperly applied to
BrowserView
s on Windows.