1750234 - Ensure Surfaces get reallocated after GPU process crash on Android

Status: RESOLVED FIXED

(GeckoView Graveyard :: Sandboxing, task, P3)

Description

[Jamie Nicol [:jnicol]]

Spin off from bug 1742569. That one landed basic support for the SurfaceAllocator to exist in the GPU process, allowing video and webgl to work on Android with the GPU process enabled.

This bug will add support for handling a GPU process crash. We need the SurfaceAllocator to reconnect to the new process, and video and webgl to gracefully reallocate their surfaces

Comment 1

[Jamie Nicol [:jnicol]]

Attachment: Bug 1750234 - Handle GPU process crash when rendering video and webgl on Android. r?agi,jgilbert

Make the SurfaceAllocator save a reference to all Surfaces it has
allocated. When the connection to the remote allocator dies, mark all
of those surfaces as released.

Make GLScreenBuffer (webgl) and RemoteVideoDecoder (video) detect when
the Surfaces they are outputting into have been released, and allocate
new ones. For video decoding on Android Marshmallow and later, we can
use MediaCodec.setOutputSurface to swap the output surface
dynamically. On older Android versions we must reconfigure the codec.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/29df96468463
Handle GPU process crash when rendering video and webgl on Android. r=jgilbert,jolin

Comment 3

[Cristina Cozmuta (:CrissCozmuta)]

https://hg.mozilla.org/mozilla-central/rev/29df96468463

Comment 4

[Chris Peterson [:cpeterson]]

Moving GPU process bugs to the new GeckoView::Sandboxing component.