-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Add setting to allow separate search engine usage for Private Window #23091
Comments
@rebron Should we maintain below UI (ddg toggle button) for some existing users? |
Initial default search provider:
|
Yes you're correct. And will be addressed in #22566. |
fix brave/brave-browser#23091 Search engines setting has separate option for configuring search provider for private profile.
fix brave/brave-browser#23091 Search engines setting has separate option for configuring search provider for private profile.
fix brave/brave-browser#23091 Search engines setting has separate option for configuring search provider for private profile.
fix brave/brave-browser#23091 Search engines setting has separate option for configuring search provider for private profile.
fix brave/brave-browser#23091 Search engines setting has separate option for configuring search provider for private profile.
Verification
|
Brave | 1.42.62 Chromium: 103.0.5060.114 (Official Build) beta (x86_64) |
---|---|
Revision | a1c2360c5b02a6d4d6ab33796ad8a268a6128226-refs/branch-heads/5060@{#1124} |
OS | macOS Version 13.0 (Build 22A5295i) |
Case one: new profile - PASSED
- install
1.42.62
- launch Brave
- open
brave://settings/search
and checkBrave
is the listed provider for bothNormal Window
andPrivate Window
- expand the list and ensure search-engine order and names match the spec (they should be the same between both window types)
- change
Brave
toQwant
forPrivate Window
- click on the "hamburger" menu and choose
New Private Window
- perform a search and confirm you're using
Qwant
- perform a search using a
Normal Window
and confirm it usesBrave
- perform a search using a
Private Window with Tor
and confirm it usesBrave
- perform a search using a
Guest Window
and confirm it usesBrave
- opened
brave://settings/searchEngines
- clicked on the
pencil
icon to editBrave
- changed
Brave
toBravest
- clicked
Save
- confirm the name has been updated in both
Window
options list - open
brave://settings/reset
- click on
Restore settings to their original defaults
- confirm by clicking on
Reset settings
- confirm default search providers have been restored
step 4 | step 5 | step 7 | step 8 | step 9 | step 10 | step 15 | step 19 |
---|---|---|---|---|---|---|---|
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
Case two: migrations - PASSED
From 1.41.96
to 1.42.62
- N/A
From 1.30.89
-> 1.40.113
-> 1.42.62
- PASSED
- install
1.30.89
- launch Brave
- open a
New Private Window
- toggle
Search with DuckDuckGo
toOn
- shut down Brave
- install/upgrade to
1.40.113
- open a
New Private Window
- confirm
Search with DuckDuckGo
is toggled - perform a search and ensure DuckDuckGo is used
- (on macOS) rename
Brave-Browser
toBrave-Browser-Beta
- install/upgrade to
1.42.62
- launch Brave
- confirm
DuckDuckGo
is chosen forPrivate Window
inbrave://settings/search
- perform a search and ensure DuckDuckGo is (still) used
step 4 | step 8 | step 9 | step 13 | step 14 |
---|---|---|---|---|
![]() |
![]() |
![]() |
![]() |
![]() |
From 1.30.89
-> 1.41.96
-> 1.42.62
- PASSED
From 1.30.89 (part A, -> 1.41.96)
- install
1.30.89
- launch Brave
- open a
New Private Window
- toggle
Search with DuckDuckGo
toOn
- shut down Brave
- trigger an update via
brave://settings/help
- confirm you get
1.41.96
- open a
New Private Window
- confirm you see
Search with DuckDuckGo
toggled toOn
- perform a search and confirm you are using
DuckDuckGo
step 4 | step 7 | step 9 | step 10 |
---|---|---|---|
![]() |
![]() |
![]() |
![]() |
(continued...)
From 1.41.96 (part B, -> 1.42.62)
- (on macOS) rename
Brave-Browser
toBrave-Browser-Beta
- launch
1.42.62
- open a
New Private Window
- perform a search and confirm you are (still) using
DuckDuckGo
- confirm
DuckDuckGo
is selected forPrivate Window
onbrave://settings/search
step 14 | step 15 |
---|---|
![]() |
![]() |
Case three: Sync - PASSED
Shared step:
- install
1.42.62
on one desktop (A) - install
1.42.62
on another desktop (B)
desktop (A)
- open
brave://settings/search
- change
Normal Window
fromBrave
toStartpage
- change
Private Window
fromBrave
toEcosia
- click on the "hamburger" menu, and then
Sync
- click on
Start using sync
- click on
Start a new Sync Chain
- click on
Computer
- under
Sync Settings
, click on theSync everything
toggle - copy the codewords
steps 2 & 3 | step 7 | step 8 |
---|---|---|
![]() |
![]() |
![]() |
desktop (B)
- click on the "hamburger" menu, and then
Sync
- click on
Start using sync
- click
I have a Sync Code
- enter/paste the codewords from desktop (A)
- click
Confirm
- click
OK
- under
Sync Settings
, click on theSync everything
toggle - open
brave://settings/search
- confirm
Normal Window
isStartpage
- confirm
Private Window
isEcosia
step 4 | step 7 | steps 9 & 10 |
---|---|---|
![]() |
![]() |
![]() |
Verification
Filed #24284 Case one: new profile -
|
Step 3 | Step4(NW) | Step4(PW) | Step 5 | Step 7 | Step8 | Step 9 | Step 10 | Step 12 | Step 15 | Step 19 |
---|---|---|---|---|---|---|---|---|---|---|
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
Case two: migrations
without modifying Preferences
; tested below
From 1.41.100
to 1.42.73
- PASSED
- install
1.41.100
- launch Brave
- open
brave://settings/search
show Normal window search engine entry - Rename
Brave-Browser
toBrave-Browser-Beta
- install
1.42.73
- launch Brave
- confirmed new entry for
Brave
is a default search provider forPrivate Window
and user will be able to select search provider via settings inbrave://settings/search
- perform a search using a
Private Window
and confirmed it usesBrave
Step 1 | Step 3 | Step 4 | Step 6 | Step 7a | Step 7b |
---|---|---|---|---|---|
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
From 1.30.89
-> 1.40.113
-> 1.42.73
- PASSED
- install
1.30.89
- launch Brave
- open a
New Private Window
- toggle
Search with DuckDuckGo
toOn
- shut down Brave
- install/upgrade to
1.40.113
- open a
New Private Window
- confirm
Search with DuckDuckGo
setting is retained and is toggled ON - perform a search and ensure DuckDuckGo is used
- Rename
Brave-Browser
toBrave-Browser-Beta
- install/upgrade to
1.42.73
- launch Brave
DuckDuckGo
toggle button is not visible anymore in private NTP- confirm
DuckDuckGo
is chosen forPrivate Window
inbrave://settings/search
- perform a search and ensure DuckDuckGo is (still) used
- Normal window still should show
Google
as a default selection
Step1 | Step 4A | Step4B | Step6 | Step 8 | Step 9 | Step 13 | Step 14 | Step 15 |
---|---|---|---|---|---|---|---|---|
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
From 1.30.89
-> 1.41.96
-> 1.42.73
- PASSED
From 1.30.89 (part A, -> 1.41.100)
- install
1.30.89
- launch Brave
- open a
New Private Window
- toggle
Search with DuckDuckGo
toOn
- trigger an update via
brave://settings/help
- confirm you get
1.41.100
- open a
New Private Window
- confirm you see
Search with DuckDuckGo
toggled toOn
- perform a search and confirm you are using
DuckDuckGo
Step1 | Step4A | Step 4B | Step5 | Step6 | Step 8 | Step 9 |
---|---|---|---|---|---|---|
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
![]() |
(continued...)
From 1.41.96 (part B, -> 1.42.73)
- Rename
Brave-Browser
toBrave-Browser-Beta
- launch
1.42.73
- open a
New Private Window
- confirmed DDG toggle button is not visible anymore in private NTP
- perform a search using a
Private Window
and confirm it usesDDG
- confirmed user will be to select search provider for private profile via settings in
brave://settings/search
Step 12 | Step 15 | Stept 16 |
---|---|---|
![]() |
![]() |
![]() |
Case three: Sync - PASSED
Verified on two profiles on the same desktop:
- profile 1
- create another profile
profile 2-sync
**profile 1 **
- open
brave://settings/search
- change
Normal Window
fromBrave
toEcosia
- change
Private Window
fromBrave
toBing
- click on the "hamburger" menu, and then
Sync
- click on
Start a new Sync Chain
- click on
Computer
- copy the codewords
- under
Sync Settings
, click on the `Sync everything'
steps 2 & 3 | step 7 | Step 8 |
---|---|---|
![]() |
![]() |
![]() |
profile 2-sync
- click on the "hamburger" menu, and then
Sync
- click
I have a Sync Code
- enter/paste the codewords from
profile 1
- click
Confirm
- click
OK
- under
Sync Settings
, click on theSync everything
toggle - open
brave://settings/search
- confirm
Normal Window
isEcosia
- confirm
Private Window
isBing
Step 5 | Step 7 | Step 8&9 |
---|---|---|
![]() |
![]() |
![]() |
Verification passed on
Brave | 1.42.81 Chromium: 104.0.5112.69 (Official Build) (64-bit) |
---|---|
Revision | 7ce2902023c722af8564068e6b26e934b83fd774-refs/branch-heads/5112@{#1213} |
OS | Ubuntu 18.04 LTS |
Case one: new profile - PASSED
- install
1.42.x
- launch Brave
- open
brave://settings/search
and checkBrave
is the listed provider for bothNormal Window
andPrivate Window
- expand the list and ensure search-engine order and names match the spec (they should be the same between both window types)
- change
Brave
toStartPage
forPrivate Window
- click on the "hamburger" menu and choose
New Private Window
- perform a search and confirm you're using
Qwant
- perform a search using a
Normal Window
and confirm it usesBrave
- perform a search using a
Private Window with Tor
and confirm it usesBrave
- perform a search using a
Guest Window
and confirm it usesBrave
- opened
brave://settings/searchEngines
- Click three dots (...) for Ecosia
- Delete
Ecosia
- confirm lists have been updated in both
Window
options list - open
brave://settings/reset
- click on
Restore settings to their original defaults
- confirm by clicking on
Reset settings
- confirm default search providers have been restored
Case two: migrations
without modifying Preferences
; tested below
From 1.41.x
to 1.42.x
- PASSED
- install
1.41.x
- launch Brave
- open
brave://settings/search
show Normal window search engine entry - Rename
Brave-Browser
toBrave-Browser-Beta
- install
1.42.x
- launch Brave
- confirmed new entry for
Brave
is a default search provider forPrivate Window
and user will be able to select search provider via settings inbrave://settings/search
- perform a search using a
Private Window
and confirmed it usesBrave
Case three: Sync - PASSED
Verified on two profiles on the same desktop:
- profile 1
- create another profile
profile 2-sync
**profile 1 **
- open
brave://settings/search
- change
Normal Window
fromBrave
toEcosia
- change
Private Window
fromBrave
toBing
- click on the "hamburger" menu, and then
Sync
- click on
Start a new Sync Chain
- click on
Computer
- copy the codewords
- under
Sync Settings
, click on the `Sync everything'
profile 2-sync
- click on the "hamburger" menu, and then
Sync
- click
I have a Sync Code
- enter/paste the codewords from
profile 1
- click
Confirm
- click
OK
- under
Sync Settings
, click on theSync everything
toggle - open
brave://settings/search
- confirm
Normal Window
isEcosia
- confirm
Private Window
isBing
Description
Users should be able to specify their search engine of choice in Private Windows. In brave://settings/search, add a new setting under "Search engine used in address bar" to allow users to use a different search engine in Private Windows separate from Normal Windows.
The list for Private Windows should be the same as Normal Windows as specified per locale.
The default for Private Windows should be Brave Search for all regions.
Expected Result
Design:
#23091 (comment)
Brave version (brave://version info)
Targeted for 1.42.x
Version/Channel Information:
Other Additional Information:
Miscellaneous Information:
Related issue: #22566
This setting already exists for Android and iOS and should be syncable.
The text was updated successfully, but these errors were encountered: