-
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
Update search engine defaults for CA / DE / FR / GB / US #18331
Comments
iOS ticket tracked here |
Adding |
Removing |
Verified passed with
Verified brave/brave-core#10326 (comment) for US region - Clean Install - PASSED
US region - Upgrade (Default SE not changed) - PASSEDInstalled
US region - Upgrade (Default SE changed) - PASSEDInstalled
GB region - Clean Install - PASSED
GB region - Upgrade (Default SE not changed) - PASSEDInstalled
GB region - Upgrade (Default SE changed) - PASSEDInstalled
Confirmed for Spain region - Clean Install - PASSED
Russia region - Clean Install - PASSED
Verified
|
Brave | 1.31.84 Chromium: 95.0.4638.40 (Oficjalna wersja) (64-bitowa) |
---|---|
Wersja | e3e7c76ba0284b16087cf4cf3153abfaef6470c7-refs/branch-heads/4638@{#624} |
System operacyjny | Linux |
Verified brave/brave-core#10326 (comment) for DE
and FR
regions.
DE region - Clean Install - PASSED
- ensured that the search section under
brave://welcome
was removed - ensured that Brave is the default SE on a clean profile
- ensured that Brave appears as the default SE under
brave://settings/search
Example | Example |
---|---|
![]() |
![]() |
DE region - Upgrade (Default SE not changed) - PASSED
Installed 1.30.89 Chromium: 94.0.4606.81
and ensured that the default SE is set as Google
Upgraded to 1.31.84 Chromium: 95.0.4638.40
and ensured that Google
was still the default SE after upgrading
Example | Example |
---|---|
![]() |
![]() |
DE region - Upgrade (Default SE changed) - PASSED
Installed 1.30.89 Chromium: 94.0.4606.81
and set the SE to be Startpage
from brave://welcome
Upgraded to 1.31.84 Chromium: 95.0.4638.40
and ensured that Startpage
was still the SE after upgrading
Example | Example |
---|---|
![]() |
![]() |
FR region - Clean Install - PASSED
- ensured that the search section under
brave://welcome
was removed - ensured that Brave is the default SE on a clean profile
- ensured that Brave appears as the default SE under
brave://settings/search
Example | Example |
---|---|
![]() |
![]() |
FR region - Upgrade (Default SE not changed) - PASSED
Installed 1.30.89 Chromium: 94.0.4606.81
and ensured that the default SE is set as Qwant
Upgraded to 1.31.84 Chromium: 95.0.4638.40
and ensured that Qwant
was still the default SE after upgrading
Example | Example |
---|---|
![]() |
![]() |
FR region - Upgrade (Default SE changed) - PASSED
Installed 1.30.89 Chromium: 94.0.4606.81
and set the SE to be Bing
from brave://welcome
Upgraded to 1.31.84 Chromium: 95.0.4638.40
and ensured that Bing
was still the SE after upgrading
Example | Example |
---|---|
![]() |
![]() |
Confirmed for Spain
and Russia
regions that SE is not set to Brave
on clean install:
Spain region - Clean Install - PASSED
- ensured that the search section under
brave://welcome
was not removed - ensured that Brave is not the default SE on a clean profile
- Saw that Brave appears as an option on
brave://welcome
andbrave://settings/search
(this is OK perBrave
appears under on-boarding viabrave://welcome
for locales other than CA / DE / FR / GB / US #18720 (comment))
Example | Example | Example |
---|---|---|
![]() |
![]() |
![]() |
Russia region - Clean Install - PASSED
- ensured that the search section under brave://welcome was not removed
- ensured that Brave is not the default SE on a clean profile, Yandex is
- Saw that Brave appears as an option on brave://welcome and brave://settings/search (this is ok per
Brave
appears under on-boarding viabrave://welcome
for locales other than CA / DE / FR / GB / US #18720 (comment)) - confirmed that Brave is not first in the list, Yandex is
Example | Example | Example |
---|---|---|
![]() |
![]() |
![]() |
Verification passed on
Brave | 1.31.84 Chromium: 95.0.4638.40 (Official Build) (64-bit)
-- | --
Revision | e3e7c76ba0284b16087cf4cf3153abfaef6470c7-refs/branch-heads/4638@{#624}
OS | Windows 10 Version 21H1 (Build 19043.1266)
Verified brave/brave-core#10326 (comment) for CA
region
CA region - Clean Install - PASSED
- ensured that the search section under brave://welcome was removed
- ensured that Brave is the default SE on a clean profile
- ensured that Brave appears as the default SE under brave://settings/search
Example | Example |
---|---|
![]() |
![]() |
CA region - Upgrade (Default SE not changed) - PASSED
Installed 1.30.89 Chromium: 94.0.4606.81
and ensured that the default SE is set as Google
Upgraded to 1.31.84 Chromium: 95.0.4638.40
and ensured that Google
was still the default SE after upgrading
Example | Example |
---|---|
![]() |
![]() |
CA region - Upgrade (Default SE changed) - PASSED
Installed 1.30.89 Chromium: 94.0.4606.81
and set the SE to be Startpage
from brave://welcome
Upgraded to 1.31.84 Chromium: 95.0.4638.40
and ensured that Startpage
was still the SE after upgrading
Example | Example |
---|---|
![]() |
![]() |
Confirmed for India
region the SE is not set to Brave
on clean install:
@brave/legacy_qa as discussed with @bsclifton, the full verifications were done on Nightly on |
Verification passed on Samsung Tab A with Android 10 running 1.31.86 x64 build
Region: US
Install type: Upgrade
Upgrade screenshots
Region: France (FR)
Install type: Upgrade
Upgrade screenshots
Regions: India
Verification passed on Oppo Reno 5 with Android 11 running 1.31.86 x64 build
Region: UK
Install type: Upgrade
Upgrade screenshots
Region: Germany (DE)
Install type: Upgrade
Upgrade screenshots
Regions: India
|
What is the rationale for this? The provided link for details doesn't work |
Test plan
Region set to CA / DE / FR / GB / US
Default search engines
listRegion set to anything else
Default search engines
list - unless you're in a region where Yandex is default (AM, AZ, BY, KG, KZ, MD, RU, TJ, TM, UZ). In that case it will be secondDescription
Update search engine defaults for:
Brave should be at the top of drop down list at brave://settings/search
Please see details here: https://github.com/brave/internal/issues/816
The text was updated successfully, but these errors were encountered: