Open Bug 1504764 Opened 1 year ago Updated 1 year ago

Visiting yandex.ru multiple times creates a top site shortcut for yandex.com

Categories

(Firefox :: New Tab Page, enhancement, P1)

enhancement

Tracking

()

ASSIGNED
Iteration:
68.3 - Apr 15 - 28
Tracking Status
firefox65 --- wontfix
firefox66 --- fix-optional

People

(Reporter: mkaply, Assigned: mkaply, NeedInfo)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

If you create a new profile and then visit yandex.ru and do some searches, when you restart you have a yandex alias.

When you click on this alias, it goes to yandex.com, not yandex.ru.

In addition, delete from history doesn't remove the alias because it's trying to remove yandex.com URLs, not yandex.ru URLs.
So the problem is here:

https://searchfox.org/mozilla-central/source/toolkit/modules/NewTabUtils.jsm#1185

This code creates a shortURL for yandex.ru (yandex) and then passes it to getSearchProvider which returns yandex.com based on the list here:

https://searchfox.org/mozilla-central/source/browser/components/newtab/lib/SearchShortcuts.jsm#14

We shouldn't be converting search URL domains to other search URL domains We should just be using the original domain and stripping off the rest of the URLs.

There's no concept of a canonical URL - every search engine has a domain specific engine.
> every search engine has a domain specific engine.

Should be:

Most search engines have country specific engines.
Assignee: nobody → mozilla
Iteration: --- → 65.2 (Nov 16)
Priority: -- → P2
Priority: P2 → P1
Iteration: 65.2 (Nov 16) → 65.3 (Nov 30)
Iteration: 65.3 (Nov 30) → 66.2 - Dec 24 - Jan 6
Iteration: 66.2 - Dec 24 - Jan 6 → 66.3 - Jan 7 - 20
Iteration: 66.3 - Jan 7 - 20 → 67.1 - Jan 28 - Feb 10
Iteration: 67.1 - Jan 28 - Feb 10 → 67.3 - Feb 25 - Mar 10
Iteration: 67.3 - Feb 25 - Mar 10 → 67.4 - Mar 11 - 17
Blocks: 1479806
Iteration: 67.4 - Mar 11 - 17 → 68.1 - Mar 18 - 31
Iteration: 68.1 - Mar 18 - 31 → 68.2 - Apr 1 - 14

This patch is simply waiting for review from Mardak and is ready to go in.

Status: NEW → ASSIGNED
Depends on: 1480888
Iteration: 68.2 - Apr 1 - 14 → 68.3 - Apr 15 - 28

There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:mkaply, could you have a look please?
For more information, please visit auto_nag documentation.

Flags: needinfo?(mozilla)
Component: Activity Streams: Newtab → New Tab Page
See Also: → 1562843
You need to log in before you can comment on or make changes to this bug.