-
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
Upgrade from Chromium 123 to Chromium 124 #36346
Comments
Verification
Filed following bugs: Site settings page is not missing any permissions - PASSEDConfirmed permissions show and expiration work as expected as below
Shields functionality to disable scripts and enable individual scripts works-PASSED
Clear browsing data works on Leo chat data - PASSEDNote: `Clear browsing data` doesn't work as there is no option in `brave://settings/clearBrowserData`. It should be `Clear Leo Data` from `brave://settings/leo-assistant` and was confirmed by @mkarolin
2024-04-10_10h47_38.mp4Avatar toolbar button looks/works as expected - PASSED
Confirmed Avatar toolbar button worked as expected
Settings -> Web3 -> Wallet Networks -> Add: search field works correctly - PASSEDConfirmed able to search and Add network via
Shields and VPN panels buttons work correctly to show the panel - PASSEDShields panel
VPN
Note: Below issues exist in
Dropping multiple urls or files onto the tabstrip works (regular + vertical) - PASSED
Bookmarks and reading list colors look ok (there are some changes in cr124) - PASSED_Bookmarks - PASSED Sidebar
Bookmarks dialog
Bookmarks bar
All bookmarks
brave://bookmarks
Reading List - PASSED
SpeedReader icon/button works correctly - PASSED
|
Verified on
Wallet (double check there is no regression)ACTUAL RESULTS:
Bottom toolbar (double check there is no regression, especially on scroll)ACTUAL RESULTS:
2024-04-09_17-50-57.mp4Sync (double check QR code works)ACTUAL RESULTS:
Search widget (double check it works as expected)ACTUAL RESULTS:
Autofill address (double check no regression there)ACTUAL RESULTS:
|
Verification
|
1.66.74 |
1.66.57 |
---|---|
No new settings
Tabs / Sidebar ✅
1.66.74 |
1.66.57 |
---|---|
No new settings
Content ✅
Shields ✅
1.66.74 |
1.66.57 |
---|---|
No new settings
NOTE: per #36859, settings have been moved around
NOTE: Block fingerprinting
preference is now an On
/Off
toggle
Privacy and security ✅
1.66.74 |
1.66.57 |
---|---|
No new settings
NOTE: per #36859, settings have been moved around
NOTE: WebRTC IP handling policy
moved below Clear browsing data
Tor windows ✅
Web3 ✅
Wallet ✅
IPFS / Web3 domains ✅
Leo ✅
Sync ✅
Search engine ✅
Extensions ✅
Autofill and passwords ✅
Languages ✅
Downloads ✅
Help tips ✅
System ✅
Reset settings ✅
---Encountered/logged:
The above requires |
Verified with
Encountered #37515 and #37517 while testing. Site settings page is not missing any permissions - PASSEDConfirmed site settings page (accessed via "tune" icon in URL bar) displays as expected.
Verify that `navigator.brave.isBrave()` works (from the DevTools console) - PASSEDConfirmed returned the same as
Shields functionality to disable scripts and enable individual scripts works - PASSED
Clear browsing data works on Leo chat data - PASSEDNote:
Avatar toolbar button looks/works as expected - PASSED
Confirmed Avatar toolbar button worked as expected
Settings -> Web3 -> Wallet Networks -> Add: search field works correctly - PASSEDConfirmed able to search and Add network via
Shields and VPN panels buttons work correctly to show the panel - PASSEDShields panel
VPN
Dropping multiple urls or files onto the tab strip works (regular + vertical) - PASSED
Ad notifications look correct (no layout anomalies) - PASSED
Bookmarks and reading list colors look ok - PASSEDBookmarks Sidebar
Bookmarks dialog
Bookmarks bar
All bookmarks
brave://bookmarks
Reading List
SpeedReader icon/button works correctly - PASSED
|
Verification
|
1.64.122 |
1.65.110 |
---|---|
- Spot-checked that if permissions were updated, they were reflected on this page.
Example | Example | Example |
---|---|---|
- Spot-checked that if a site uses cookies, they are displayed when clicking on
Cookies and Site Data
from the site settings page.
Example | Example
--- | --- | ---
Shields functionality to disable scripts and enable individual scripts works - PASSED
example | example | example | example | example | example | example |
---|---|---|---|---|---|---|
Settings -> Web3 -> Wallet Networks -> Add: search field works correctly - PASSED
example | example | example | example | example | example | example |
---|---|---|---|---|---|---|
Shields and VPN panels buttons work correctly to show the panel - PASSED
Shields
example | example | example |
---|---|---|
VPN
Unavailable on linux
.
Dropping multiple urls or files onto the tabstrip works (regular + vertical) - PASSED
example | example | example | example | example | example |
---|---|---|---|---|---|
Ad notifications look correct (no layout anomalies) - SKIPPING for now due to VM
Bookmarks and reading list colors look ok (there are some changes in cr124) - PASSED
example | example | example |
---|---|---|
Encountered:
Labelling as |
Verified on
Wallet
Sync
|
Upgrade from Chromium 123 to Chromium 124
https://chromium.googlesource.com/chromium/src/+log/123.0.6312.4...124.0.6315.0/?pretty=fuller&n=10000
QA tests:
This is a major Chromium version bump, please do full passes.
Desktop Affected areas:
navigator.brave.isBrave()
works (from the DevTools console).iOS Affected areas:
Android Affected areas:
The text was updated successfully, but these errors were encountered: