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: ensure chrome colors are initialized #35034
Conversation
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.
There is linking error on Windows
..\..\third_party\llvm-build\Release+Asserts\bin\lld-link.exe /OUT:./electron.exe /nologo -libpath:..\..\third_party\llvm-build\Release+Asserts\lib\clang\15.0.0\lib\windows "-libpath:../../../../Program Files (x86)/Microsoft Visual Studio/2019/Community/VC/Tools/MSVC/14.29.30133/ATLMFC/lib/x64" "-libpath:../../../../Program Files (x86)/Microsoft Visual Studio/2019/Community/VC/Tools/MSVC/14.29.30133/lib/x64" "-libpath:../../../../Program Files (x86)/Windows Kits/NETFXSDK/4.8/lib/um/x64" "-libpath:../../../../Program Files (x86)/Windows Kits/10/lib/10.0.20348.0/ucrt/x64" "-libpath:../../../../Program Files (x86)/Windows Kits/10/lib/10.0.20348.0/um/x64" /MACHINE:X64 /PDB:./electron.exe.pdb @./electron.exe.rsp
lld-link: error: undefined symbol: bool __cdecl ShouldCustomDrawSystemTitlebar(void)
The missing definition looks being defined here: |
Thanks @MarshallOfSound! |
Thank you @MarshallOfSound! |
I think we need to add |
@MarshallOfSound, sorry for the ping, could you apply the suggestions we wrote, if you have your hands full of work I can try to do a similar PR trying to address this problem (I know nothing C++ code but I can follow your code and use the suggestions here) just let me know. |
54d5b4f
to
4ac134a
Compare
4ac134a
to
bfcdf1b
Compare
7f1cfa7
to
dfc3fb1
Compare
Release Notes Persisted
|
I have automatically backported this PR to "19-x-y", please check out #35400 |
I have automatically backported this PR to "21-x-y", please check out #35401 |
I have automatically backported this PR to "20-x-y", please check out #35402 |
/trop run backport-to 18-x-y |
@arnaudbud is not authorized to run PR backports. |
@arnaudbud I have tested 18.x it there is no such issue |
* fix: ensure chrome colors are initialized * build: fix linking on windows * build: fix linking on windows * build: add needed files to chromium_src/BUILD Co-authored-by: VerteDinde <keeleymhammond@gmail.com>
Fixes #34406
After https://chromium-review.googlesource.com/c/chromium/src/+/3526815 we need to initialize chromes colors if we want to support pip.
Notes: Picture-In-Picture mode no longer becomes a red rectangle on hover