-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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 98.0.4758.66 to Chromium 98.0.4758.72. #20726
Comments
Remaining items:
cc @brave/legacy_qa ☝🏻 |
Verification passed on the following devices running 1.35.97 x64 build
SKU tests verified here #20741 (comment) |
Verification
|
Brave | 1.35.97 Chromium: 98.0.4758.72 (Official Build) (x86_64) |
---|---|
Revision | d63e30f73734c5326037b0d94eceed77168d95e1-refs/branch-heads/4758@{#905} |
OS | macOS Version 11.6.1 (Build 20G224) |
- Confirmed all the settings are displayed as expected in
brave://settings
and UI also looks good. - Confirmed content settings are displayed as expected in
brave://settings/content
- Confirmed security settings are as expected, no new settings are added as part of CR 98 in
brave://settings/security
- Confirmed
brave://settings/searchEngines
is displayed as expected no new settings added, the UI looks good - Confirmed all the settings under
brave://settings/extensions
are displayed as expected - Confirmed
Share this page
options works as expected
Encountered #20750 while testing.
- Confirmed clicking on
Show Brave News
shows the Brave News feeds - Confirmed content cards are loading the images correctly in feeds when the Brave News feed is scrolled down
- Confirmed click on any image/link on news feed redirects to correct target link
- Confirmed images render properly on brave news feeds when any publisher is enabled/disabled via
Sources
- Cosmetic filtering works as expected
- Element on any page can be blocked via block element selector
element blocked | blocked rule | remove rules | blocked elements reappeared |
---|---|---|---|
- Encountered the issue Regional adblock filters are not pre-selected based on locale #4367
- Launched browser using
--lang=fr
and confirmed correct Adblock regional filter is pre-selected inbrave://adblock
- Launched browser using
--lang=de
and confirmed correct Adblock regional filter is pre-selected inbrave://adblock
Example | Example |
---|---|
- Confirmed Tab search option is shown when the
Show tab search button
settings is enabled - Confirmed able to search the particular tab from list of tabs
- Confirmed Tab search option isn't shown when the
Show tab search button
settings is disabled - Confirmed
Show tab search button
setting retained after browser restart
Example | Example | Example |
---|---|---|
Followed the steps for the two SKU testcases over in brave/brave-core#11985 (comment)
Case 1: Verify SKU functionality is disable on allow-list domains
- Visit https://account.brave.software and enter basic auth credentials (reach out to @bsclifton for details)
- Open dev tools (F12, etc) and go to
Console
tab - Paste in:
await window.chrome.braveSkus.refresh_order('0ee1e3ee-afc3-4f2f-b2a2-3d1e401e1530')
and hit enter. It should showUncaught TypeError: Cannot read properties of undefined (reading 'refresh_order') at <anonymous>:1:31
- Steps 1-3 can be repeated on
https://account.brave.com
orhttps://account.bravesoftware.com
account.brave.com |
account.bravesoftware.com |
account.brave.software |
---|---|---|
Case 2: Verify SKU functionality is disabled on non-allowed domains
- Visit any non-Brave website
- Open dev tools (F12, etc) and go to
Console
tab - Paste in:
await window.chrome.braveSkus.refresh_order('0ee1e3ee-afc3-4f2f-b2a2-3d1e401e1530')
and hit enter. It should showUncaught TypeError: Cannot read properties of undefined (reading 'refresh_order') at <anonymous>:1:31
Verification
|
Brave | 1.35.97 Chromium: 98.0.4758.72 (Official Build) (64-bit) |
---|---|
Revision | d63e30f73734c5326037b0d94eceed77168d95e1-refs/branch-heads/4758@{#905} |
OS | Linux |
- Confirmed all the settings are displayed as expected in
brave://settings
and UI also looks good. - Confirmed content settings are displayed as expected in
brave://settings/content
- Confirmed security settings are as expected, no new settings are added as part of CR 98 in
brave://settings/security
- Confirmed
brave://settings/searchEngines
is displayed as expected no new settings added, the UI looks good - Confirmed all the settings under
brave://settings/extensions
are displayed as expected - Confirmed
Share this page
options works as expected
Encountered #20750 while testing.
- Confirmed clicking on
Show Brave News
shows the Brave News feeds - Confirmed content cards are loading the images correctly in feeds when the Brave News feed is scrolled down
- Confirmed click on any image/link on news feed redirects to correct target link
- Confirmed images render properly on brave news feeds when any publisher is enabled/disabled via
Sources
- Cosmetic filtering works as expected
- Element on any page can be blocked via block element selector
element blocked | blocked rule | remove rules | blocked elements reappeared |
---|---|---|---|
- Encountered the issue Regional adblock filters are not pre-selected based on locale #4367
- Launched browser using
--lang=fr
and confirmed correct Adblock regional filter is pre-selected inbrave://adblock
- Launched browser using
--lang=de
and confirmed correct Adblock regional filter is pre-selected inbrave://adblock
Example | Example |
---|---|
- Confirmed Tab search option is shown when the
Show tab search button
settings is enabled - Confirmed able to search the particular tab from list of tabs
- Confirmed Tab search option isn't shown when the
Show tab search button
settings is disabled - Confirmed
Show tab search button
setting retained after browser restart
Example | Example | Example |
---|---|---|
Followed the steps for the two SKU testcases over in brave/brave-core#11985 (comment)
Case 1: Verify SKU functionality is disable on allow-list domains
- Visit https://account.brave.software and enter basic auth credentials (reach out to @bsclifton for details)
- Open dev tools (F12, etc) and go to
Console
tab - Paste in:
await window.chrome.braveSkus.refresh_order('0ee1e3ee-afc3-4f2f-b2a2-3d1e401e1530')
and hit enter. It should showUncaught TypeError: Cannot read properties of undefined (reading 'refresh_order') at <anonymous>:1:31
- Steps 1-3 can be repeated on
https://account.brave.com
orhttps://account.bravesoftware.com
account.brave.com |
account.bravesoftware.com |
account.brave.software |
---|---|---|
Case 2: Verify SKU functionality is disabled on non-allowed domains
- Visit any non-Brave website
- Open dev tools (F12, etc) and go to
Console
tab - Paste in:
await window.chrome.braveSkus.refresh_order('0ee1e3ee-afc3-4f2f-b2a2-3d1e401e1530')
and hit enter. It should showUncaught TypeError: Cannot read properties of undefined (reading 'refresh_order') at <anonymous>:1:31
labelling |
Minor Chromium bump.
https://chromium.googlesource.com/chromium/src/+log/98.0.4758.66..98.0.4758.72?pretty=fuller&n=10000
QA tests:
Check branding items
Check for version bump
Additional checks:
No specific code changes in Brave (only line number changes in patches).
The text was updated successfully, but these errors were encountered: