1640292 - Move RemoteSettingsWorker.checkContentHash from worker to main thread

Status: RESOLVED FIXED

(Firefox :: Remote Settings Client, enhancement)

Description

[Rob Wu [:robwu]]

The attachment downloader relies on RemoteSettingsWorker.checkContentHash to check the integrity of an ArrayBuffer. This is implemented by posting the data to a worker thread, using the web crypto API to compute the hash and then posting the result back.

The web crypto API already uses a thread pool, so there is no benefit from passing the buffer to a separate worker thread before computing the digest.

Let's do the check from the main thread instead:

• In the best case, when the sizes don't match, there is almost no cost of running the check on the main thread.
• In the worst case, when the hash has to be computed, the crypto API will process the data on a separate thread, which is not more expensive than passing the data to a separate thread first.

Comment 1

[Rob Wu [:robwu]]

Linking to bug 1508332 for visibility. Both bugs share the same goal (improving performance), but do so in a different way: the other bug suggests to move operations to a worker thread, whereas this bug is about moving an operation from the worker thread to the main thread.

The work in this bug does not preclude the work in bug 1508332 ; it is possible to move this function together with all of its callers to a worker thread. This bug is just concerned with ensuring that checksum calculation happens on the same thread as where the data was read.

Comment 2

[Rob Wu [:robwu]]

Attachment: Bug 1640292 - Move RemoteSettingsWorker.checkContentHash to main thread

The crypto API does most of its work on the background thread. There is
no benefit in posting the buffer to a worker thread.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ddc560ee9d02
Move RemoteSettingsWorker.checkContentHash to main thread r=leplatrem

Comment 4

[Razvan Maries]

https://hg.mozilla.org/mozilla-central/rev/ddc560ee9d02