1878219 - Move core Categorization Map into a worker

Status: RESOLVED FIXED

(Firefox :: Search, task, P2)

Description

[James Teow [:jteow]]

As we get closer to posting the category mappings, one concern is if there's a conflict between the initial startup loading the attachments and a test event forcing a sync.

To avoid work that could get overwritten anyway, we should bail in clearAndPopulateMap where it seems fitting.

Comment 1

[James Teow [:jteow]]

Attachment: Bug 1878219 - Bailout from populating domain to categories map if a more recent attempt was detected - r?scunnane!

Comment 2

[James Teow [:jteow]]

The concern about bailouts was about testing interfering with downloading content from RS, but since tests don't pull from the real RS this shouldn't be a concern.

I've been testing running the mappings from stage - preview and I am wondering if the impact to startup especially on the main thread is acceptable or if we should look to move some of this to a Worker and storing the information there.

This might make querying the map async, but unlike other consumers who have large storage requirements like Urlbar and are user facing, we don't require near instant responses.

Comment 3

[James Teow [:jteow]]

Attachment: Bug 1878219 - Move the core Categorization Map into a worker - r?scunnane!

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/904f2cb23332
Move the core Categorization Map into a worker - r=scunnane

Comment 5

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/904f2cb23332