Bug 1953677 [Linux] Delete snapshot GL context when VideoFramePool is deleted r=alwu

Differential Revision: https://phabricator.services.mozilla.com/D243289

Author: stransky

dom/media/platforms/ffmpeg/FFmpegVideoFramePool.cpp

@@ -124,8 +124,11 @@ VideoFramePool<LIBAV_VER>::VideoFramePool(int aFFMPEGPoolSize)
 }
 
 VideoFramePool<LIBAV_VER>::~VideoFramePool() {
+  DMABUF_LOG("VideoFramePool::~VideoFramePool()");
   MutexAutoLock lock(mSurfaceLock);
   mDMABufSurfaces.Clear();
+
+  DMABufSurface::DeleteSnapshotGLContext();
 }
 
 void VideoFramePool<LIBAV_VER>::ReleaseUnusedVAAPIFrames() {

widget/gtk/DMABufSurface.cpp

@@ -167,6 +167,12 @@ void ReturnSnapshotGLContext(RefPtr<GLContext> aGLContext) {
   egl->fMakeCurrent(EGL_NO_SURFACE, EGL_NO_SURFACE, EGL_NO_CONTEXT);
 }
 
+void DMABufSurface::DeleteSnapshotGLContext() {
+  StaticMutexAutoLock lock(sSnapshotContextMutex);
+  sSnapshotContext = nullptr;
+  GLContextProviderEGL::Shutdown();
+}
+
 bool DMABufSurface::UseDmaBufGL(GLContext* aGLContext) {
   static bool useDmabufGL =
       [&]() {

widget/gtk/DMABufSurface.h

@@ -184,6 +184,8 @@ class DMABufSurface {
   void GlobalRefAdd();
   void GlobalRefRelease();
 
+  static void DeleteSnapshotGLContext();
+
   // Release all underlying data.
   virtual void ReleaseSurface() = 0;