Skip to content
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

Closed
bsclifton opened this issue Sep 24, 2021 · 7 comments · Fixed by brave/brave-core#10326
Closed

Update search engine defaults for CA / DE / FR / GB / US #18331

bsclifton opened this issue Sep 24, 2021 · 7 comments · Fixed by brave/brave-core#10326

Comments

@bsclifton
Copy link
Member

bsclifton commented Sep 24, 2021

Test plan

Region set to CA / DE / FR / GB / US

  1. Fresh profile
  2. Search onboarding should NOT be shown
  3. Default search engine SHOULD be Brave
  4. Visit brave://settings/searchEngines
  5. Brave SHOULD be first in the Default search engines list

Region set to anything else

  1. Fresh profile
  2. Search onboarding SHOULD be shown
  3. Default will NOT be Brave Search
  4. Brave Search should NOT be in the Onboarding list
  5. Visit brave://settings/searchEngines
  6. Brave SHOULD be first in the 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 second

Description

Update search engine defaults for:

  • Canada (CA)
  • Germany (DE)
  • France (FR)
  • Great Britain (GB)
  • United States (US)

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

@iccub
Copy link

iccub commented Sep 24, 2021

iOS ticket tracked here
brave/brave-ios#4221

bsclifton added a commit to brave/brave-core that referenced this issue Sep 28, 2021
@rebron rebron added QA/Yes priority/P2 A bad problem. We might uplift this to the next planned release. labels Sep 28, 2021
bsclifton added a commit to brave/brave-core that referenced this issue Sep 30, 2021
@bsclifton bsclifton added this to the 1.32.x - Nightly milestone Oct 5, 2021
@kjozwiak
Copy link
Member

kjozwiak commented Oct 13, 2021

Adding QA/Blocked as the above will require a 1.31.x RC which I'm hoping to kick off tonight 👍 We'll need 1.31.84 or higher.

@LaurenWags
Copy link
Member

Removing QA/Blocked since 1.31.84 is available - https://github.com/brave/brave-browser/releases/tag/v1.31.84

@LaurenWags
Copy link
Member

LaurenWags commented Oct 13, 2021

Verified passed with

Brave | 1.31.84 Chromium: 95.0.4638.40 (Official Build) (x86_64)
-- | --
Revision | e3e7c76ba0284b16087cf4cf3153abfaef6470c7-refs/branch-heads/4638@{#624}
OS | macOS Version 10.15.7 (Build 19H1417)

Verified brave/brave-core#10326 (comment) for US and GB regions.

US 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
US welcome US settings
US 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
1 30 x G 1 31 x G
US region - Upgrade (Default SE changed) - PASSED

Installed 1.30.89 Chromium: 94.0.4606.81 and set the SE to be Ecosia from brave://welcome
Upgraded to 1.31.84 Chromium: 95.0.4638.40 and ensured that Ecosia was still the SE after upgrading

Example Example
1 30 x E 1 31 x E
GB 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
GB1 GB2
GB 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
1 30 x Goog 1 31 x Goog
GB 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
1 30 x Bing 1 31 x Bing

Confirmed for Spain and Russia regions that SE is not set to Brave on clean install:

Spain region - Clean Install - PASSED
Example Example Example
Spain-onboarding2 Spain-onboarding1 Spain-settings
Russia region - Clean Install - PASSED
Example Example Example
R1 R2 R3

Verified IN-PROGRESS with

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
Screen Shot 2021-10-13 at 1 27 49 PM Screen Shot 2021-10-13 at 1 27 43 PM
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
Screen Shot 2021-10-13 at 1 25 57 PM Screen Shot 2021-10-13 at 1 26 49 PM
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
Screen Shot 2021-10-13 at 1 32 25 PM Screen Shot 2021-10-13 at 1 32 53 PM
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
Screen Shot 2021-10-13 at 1 37 58 PM Screen Shot 2021-10-13 at 1 38 22 PM
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
Screen Shot 2021-10-13 at 1 42 52 PM Screen Shot 2021-10-13 at 1 43 18 PM
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
Screen Shot 2021-10-13 at 1 46 05 PM Screen Shot 2021-10-13 at 1 46 36 PM

Confirmed for Spain and Russia regions that SE is not set to Brave on clean install:

Spain region - Clean Install - PASSED
Example Example Example
Screen Shot 2021-10-13 at 2 07 58 PM Screen Shot 2021-10-13 at 2 08 02 PM Screen Shot 2021-10-13 at 2 08 15 PM
Russia region - Clean Install - PASSED
Example Example Example
Screen Shot 2021-10-13 at 2 04 11 PM Screen Shot 2021-10-13 at 2 04 20 PM Screen Shot 2021-10-13 at 2 04 34 PM

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
image image
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
image image
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
image image

Confirmed for India region the SE is not set to Brave on clean install:

India 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 and brave://settings/search
Example Example Example
image image image

@kjozwiak
Copy link
Member

@brave/legacy_qa as discussed with @bsclifton, the full verifications were done on Nightly on Win via brave/brave-core#10326 (comment) before uplifting the Brave Search work into 1.31.x. So in interest of time, we can just run through ~two of the locales when running through verifications rather than checking all five of the locales (CA / DE / FR / GB / US).

@stephendonner stephendonner added QA/In-Progress Indicates that QA is currently in progress for that particular issue QA Pass-Linux and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Oct 13, 2021
@srirambv
Copy link
Contributor

srirambv commented Oct 18, 2021

Verification passed on Samsung Tab A with Android 10 running 1.31.86 x64 build

Region: US
Install Type: Clean Install

Clean Install screenshots
Normal Tab SE Private Tab SE
image image

Install type: Upgrade

  • Verified upgrading from previous version retains the search engine selected
  • Verified before upgrade Brave is listed second on the list. Post upgrade, Brave is listed on top of the list
Upgrade screenshots
Normal Tab SE (before upgrade) Normal Tab SE (post upgrade) Normal Tab SE (before upgrade) Normal Tab SE (post upgrade)
image image image image

Region: France (FR)
Install Type: Clean Install

Upgrade screenshots
Normal Tab SE Private Tab SE
image image

Install type: Upgrade

  • Verified upgrading from previous version retains the search engine selected
  • Verified before upgrade Brave is listed second on the list. Post upgrade, Brave is listed on top of the list
Upgrade screenshots
Normal Tab SE (before upgrade) Normal Tab SE (post upgrade) Normal Tab SE (before upgrade) Normal Tab SE (post upgrade)
image image image image

Regions: India
Install Type: Clean Install

  • Verified onboarding is shown on clean install
  • Verified Brave is not listed as an option in the onboarding screen
  • Verified Brave is listed as the first option for both normal and private tab search engine settings

Verification passed on Oppo Reno 5 with Android 11 running 1.31.86 x64 build

Region: UK
Install Type: Clean Install

Clean Install screenshots
Normal Tab SE Private Tab SE
image image

Install type: Upgrade

  • Verified upgrading from previous version retains the search engine selected
  • Verified before upgrade Brave is listed second on the list. Post upgrade, Brave is listed on top of the list
Upgrade screenshots
Normal Tab SE (before upgrade) Normal Tab SE (post upgrade) Normal Tab SE (before upgrade) Normal Tab SE (post upgrade)
image image image image

Region: Germany (DE)
Install Type: Clean Install

Upgrade screenshots
Normal Tab SE Private Tab SE
image image

Install type: Upgrade

  • Verified upgrading from previous version retains the search engine selected
  • Verified before upgrade Brave is listed second on the list. Post upgrade, Brave is listed on top of the list
Upgrade screenshots
Normal Tab SE (before upgrade) Normal Tab SE (post upgrade) Normal Tab SE (before upgrade) Normal Tab SE (post upgrade)
image image image image

Regions: India
Install Type: Clean Install

  • Verified onboarding is shown on clean install
  • Verified Brave is not listed as an option in the onboarding screen
  • Verified Brave is listed as the first option for both normal and private tab search engine settings

@bertrandfalguiere
Copy link

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

What is the rationale for this? The provided link for details doesn't work

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

9 participants