2085189 - Make TextureFactoryIdentifier universal for all windows

Status: RESOLVED FIXED

(Core :: Graphics, task)

Description

[Andrew Osmond [:aosmond] (he/him)]

Now that we've fixed the sync object issues, and make video decoders pull from the GPU process, we can actually simplify the TextureFactoryIdentifier. Most of the fields aren't used by videos, and the ones that are can be now considered universal. There is a small bug with D3D11 compositing + SWGL that will be fixed before landing.

Comment 1

[Andrew Osmond [:aosmond] (he/him)]

Attachment: Bug 2085189 - Give remote video decoders a process-wide TextureFactoryIdentifier.

Decoders created while the GPU process is being recreated asked the
compositor for the identifier of a compositor used by their content
process, which did not exist yet. Without an identifier, the decoder
had no video bridge, and could not decode at all if it needed one, e.g.
for video with alpha. Build the identifier from the state of the
process instead, which does not depend on any particular widget.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/3ed2c1395615
https://hg.mozilla.org/integration/autoland/rev/1f3f1ca41dad
Give remote video decoders a process-wide TextureFactoryIdentifier. r=gfx-reviewers,lsalzman

Comment 3

[Iulian Moraru]

https://hg.mozilla.org/mozilla-central/rev/1f3f1ca41dad