1891982 - Duplicate TopSitesFeed and relevant tests into a separate component

Status: RESOLVED FIXED

(Firefox :: Top Sites, task, P2)

Description

[James Teow [:jteow]]

TopSitesFeed.sys.mjs will be duplicated into browser/components/topsites/TopSites.sys.mjs.

Likewise browser/components/newtab/test/xpcshell/test_TopSitesFeed.js.

The component should be called TopSites to avoid confusion with TopSitesFeed.

Comment 1

[James Teow [:jteow]]

Attachment: Bug 1891982 - Part 1: Copy Top Sites Feed into the Top Sites component - r?dao!

Comment 2

[James Teow [:jteow]]

Attachment: Bug 1891982 - Part 2: Do minimal renaming of TopSitesFeed to TopSites - r?dao!

Avoiding renaming all references to TopSitesFeed (especially in tests) until
after I remove extraneous code and refactor in stages the component code.

Depends on D207827

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/46b9f1bb340b
Part 1: Copy Top Sites Feed into the Top Sites component - r=dao,frontend-codestyle-reviewers,Standard8
https://hg.mozilla.org/integration/autoland/rev/21035f8f528d
Part 2: Do minimal renaming of TopSitesFeed to TopSites - r=dao

Comment 4

[Aron Cseh]

https://hg.mozilla.org/mozilla-central/rev/46b9f1bb340b
https://hg.mozilla.org/mozilla-central/rev/21035f8f528d