1864861 - Factor code in `FontPreloader` used by `FontFaceSetWorkerImpl` out to `FontLoaderUtils`

Status: RESOLVED FIXED

(Core :: Layout, task)

Description

[Mirko Brodesser (:mbrodesser) -- not reading bugmails currently]

Primarily about https://searchfox.org/mozilla-central/rev/7e60ac48dc5b3e3f06c1baf2a9a6e0352bd85c01/layout/style/FontPreloader.cpp#158

Reason to factor it out is workers don't preload fonts, they only load them.

Comment 1

[Mirko Brodesser (:mbrodesser) -- not reading bugmails currently]

Attachment: Bug 1864861: part 1) Factor general font loading code out to FontLoaderUtils. r=dholbert

Removes the dependency of font-face loading code to FontPreloader.
The latter's purpose is to preload fonts.

The code was just moved with includes updated.

Comment 2

[Mirko Brodesser (:mbrodesser) -- not reading bugmails currently]

Attachment: Bug 1864861: part 2) Remove `aIsPreload` argument from `FontLoaderUtils::BuildChannel` for workers. r=dholbert

Depends on D193789

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/cb42aa471521
part 1) Factor general font loading code out to FontLoaderUtils. r=jfkthame
https://hg.mozilla.org/integration/autoland/rev/aaa7a9d78543
part 2) Remove `aIsPreload` argument from `FontLoaderUtils::BuildChannel` for workers. r=dholbert

Comment 4

[Serban Stanca [:SerbanS]]

https://hg.mozilla.org/mozilla-central/rev/cb42aa471521
https://hg.mozilla.org/mozilla-central/rev/aaa7a9d78543