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 Startpage search to give attribution to Brave #12257

Closed
bsclifton opened this issue Oct 21, 2020 · 1 comment · Fixed by brave/brave-core#6916
Closed

Update Startpage search to give attribution to Brave #12257

bsclifton opened this issue Oct 21, 2020 · 1 comment · Fixed by brave/brave-core#6916

Comments

@bsclifton
Copy link
Member

bsclifton commented Oct 21, 2020

Test plan

New profile

  1. Visit brave://settings/searchEngines and set Start Page as default engine
  2. Do a search for cats in the omnibox
  3. Verify search URL is in the format like https://www.startpage.com/do/search?q=cats&segment=startpage.brave

Existing profile

  1. Use an older version of Brave
  2. Set StartPage as default (notice casing)
  3. Do a search for cats in the omnibox
  4. Verify search URL has pl=opensearch in it
  5. Upgrade to updated version
  6. Visit brave://settings/searchEngines
  7. Ensure engine is showing as Startpage and not StartPage
  8. Do a search for cats in the omnibox
  9. Verify search URL is in the format like https://www.startpage.com/do/search?q=cats&segment=startpage.brave

Description

We currently offer StartPage in our list of "Default search engines" (visible on brave://settings/searchEngines)
image

The integration here is just using the Open Search provided parameters. We should update so that Brave can be attributed for the search.

Also - the branding should be Startpage and not StartPage

Steps to Reproduce

  1. New profile
  2. Visit brave://settings/searchEngines and set Start Page as default engine
  3. Do a search for cats in the omnibox

Actual result:

Search URL is in format like https://www.startpage.com/do/dsearch?query=cats&cat=web&pl=opensearch

Expected result:

Search URL should be in the format like https://www.startpage.com/do/search?q=cats&segment=startpage.brave

@bsclifton bsclifton added feature/search OS/Android Fixes related to Android browser functionality OS/Desktop labels Oct 21, 2020
@bsclifton bsclifton self-assigned this Oct 21, 2020
bsclifton added a commit to brave/brave-core that referenced this issue Oct 21, 2020
- gives Brave attribution
- fixes branding mistake; StartPage (wrong) vs Startpage (correct)

Fixes brave/brave-browser#12257
@bsclifton bsclifton added this to the 1.18.x - Nightly milestone Oct 21, 2020
@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Oct 26, 2020

Verification is in-progress

Brave | 1.16.68 Chromium: 86.0.4240.111 (Official Build) (64-bit)
-- | --
Revision | b8c36128a06ebad76af51591bfec980224db5522-refs/branch-heads/4240@{#1290}
OS | Windows 10 OS Version 1903 (Build 18362.1139)

  • Verified the STR from the description

New profile

  • Verified that default engine cab be set to Startpage in brave://settings/searchEngines
  • Performed search cats in the omnibox and ensured search engine formate is displayed like https://www.startpage.com/do/search?q=cats&segment=startpage.brave
    image
  • Ensured default search engine is showing as Startpage in brave://settings/search
    image

Existing profile

  • Installed older version 1.12.x and set the default search engine as Startpage and performed search cats and ensures that pl=opensearch in the search bar
    image

pl=opensearch
image

  • Ensured default search engine is showing as StartPage in brave://settings/search
    image

  • Upgraded profile to 1.16.68 and ensured StartPage is displayed as Startpage in brave://settings/searchEngines
    image

  • Ensured default search engine is showing as Startpage in brave://settings/search
    image

  • Performed search cats in the omnibox and ensured search engine formate is displayed like https://www.startpage.com/do/search?q=cats&segment=startpage.brave
    image


Verified passed with

Brave	1.16.68 Chromium: 86.0.4240.111 (Official Build) (x86_64)
Revision	b8c36128a06ebad76af51591bfec980224db5522-refs/branch-heads/4240@{#1290}
OS	macOS Version 10.14.6 (Build 18G6032)
Confirmed casing on Startpage in multiple areas

brave://welcome page:

welcome

brave://settings/

settings1 settings2

Omnibox search:

omnibox

Right click search:

context
Confirmed search URL is configured as expected config
Confirmed search URL shows as expected searchURL

Verification passed on

Brave 1.16.68 Chromium: 86.0.4240.111 (Official Build) (64-bit)
Revision b8c36128a06ebad76af51591bfec980224db5522-refs/branch-heads/4240@{#1290}
OS Ubuntu 18.04 LTS
Confirmed casing on Startpage in multiple areas

brave://welcome page:
image

brave://settings/

image
image

Omnibox search:

image

Right click search:
image

Confirmed search URL is configured as expected

image

Confirmed search URL shows as expected ![image](https://user-images.githubusercontent.com/34715963/97231251-83b6bd00-17db-11eb-841b-fd2a72a555c4.png)

Verification PASSED on Samsung S10+ running Android 10 using 1.16.68 CR: 86.0.4240.111.

Ensured that the correct casing was being used throughout Brave
**`Standard Tab Default Choices`**

Screenshot_20201026-212715_Brave

Private Tab Default Choices

Screenshot_20201026-212721_Brave

SP Selected as Default

Screenshot_20201026-212731_Brave

Ensured that the correct URL segment/format is being used

Screenshot_20201026-214314_Brave


Verification passed on Brave v1.16.68 on Samsung Galaxy Tab S5e (Android 9.0)

Ensured that the correct casing was being used throughout Brave
**`Standard Tab Choices`** ![Screenshot_20201027-161824_Brave](https://user-images.githubusercontent.com/34715963/97323009-a3e48b80-1870-11eb-8571-3384f622a8e0.jpg)

Private Tab Default Choices
Screenshot_20201027-161831_Brave

SP Selected as Default

Screenshot_20201027-162134_Brave

Ensured that the correct URL segment/format is being used

Screenshot_20201027-161620_Brave

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