-
Notifications
You must be signed in to change notification settings - Fork 2.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
Remove chrome-common-chrome_content_client_constants.cc.patch #14338
Comments
This patch can be removed in favour of chromium_src overrides to redefine the values for the constants we're interested in. Resolves brave/brave-browser#14338
Verified
Test fix for brave/brave-browser#3694
NOTE: Same behavior on Test fix for brave/brave-browser#884
Test fix for brave/brave-browser#4424
Embed test (possible fix for brave/brave-browser#3119)
Test that PDF.js is still installable
Test that PDF.js does not show
Verification passed on
Test fix for brave/brave-browser#3694
Click on Test fix for brave/brave-browser#884
Test fix for brave/brave-browser#4424Embed test (possible fix for brave/brave-browser#3119)
Test that PDF.js is still installable
Test that PDF.js does not show
Installed Upgraded the profile to Verify PDF.js does NOT show in Verification PASSED on
Test fix for brave/brave-browser#3694
NOTE: Ran into #11913 when going through this case. Asking me to save the
Test fix for brave/brave-browser#884
Test fix for brave/brave-browser#4424
Embed test (possible fix for brave/brave-browser#3119)
Test that PDF.js is still installable
Test that PDF.js does not show
|
Removing |
Description
This patch could be removed in favour of chromium_src overrides to redefine the values for the constants we're interested in.
Desktop Brave version:
Brave 1.23.1 (Chromium 89.0.4389.58)
Miscellaneous Information:
This removal references a patch initially introduced with brave/brave-core#2342
The text was updated successfully, but these errors were encountered: