VaapiVideoAccelerator: make Vaapi accelerator work with ozone

content/common/gpu/media/gpu_video_decode_accelerator.cc

Index: content/common/gpu/media/gpu_video_decode_accelerator.cc
diff --git a/content/common/gpu/media/gpu_video_decode_accelerator.cc b/content/common/gpu/media/gpu_video_decode_accelerator.cc
index 025d971074f1724d700f965fe4bd914ea5a02a76..25bc52504d30013f9ec05cb145a0fda8fdf9c7a6 100644
--- a/content/common/gpu/media/gpu_video_decode_accelerator.cc
+++ b/content/common/gpu/media/gpu_video_decode_accelerator.cc
@@ -31,9 +31,8 @@
 #elif defined(OS_CHROMEOS) && defined(ARCH_CPU_ARMEL) && defined(USE_X11)
 #include "content/common/gpu/media/v4l2_video_decode_accelerator.h"
 #include "content/common/gpu/media/v4l2_video_device.h"
-#elif defined(OS_CHROMEOS) && defined(ARCH_CPU_X86_FAMILY) && defined(USE_X11)
+#elif defined(OS_CHROMEOS) && defined(ARCH_CPU_X86_FAMILY)
 #include "content/common/gpu/media/vaapi_video_decode_accelerator.h"
-#include "ui/gl/gl_context_glx.h"
 #include "ui/gl/gl_implementation.h"
 #elif defined(USE_OZONE)
 #include "media/ozone/media_ozone_platform.h"
@@ -273,26 +272,9 @@ void GpuVideoDecodeAccelerator::Initialize(
       make_context_current_,
       device.Pass(),
       io_message_loop_));
-#elif defined(OS_CHROMEOS) && defined(ARCH_CPU_X86_FAMILY) && defined(USE_X11)
-  if (gfx::GetGLImplementation() != gfx::kGLImplementationDesktopGL) {
-    VLOG(1) << "HW video decode acceleration not available without "
-               "DesktopGL (GLX).";
-    SendCreateDecoderReply(init_done_msg, false);
-    return;
-  }
-  gfx::GLContextGLX* glx_context =
-      static_cast<gfx::GLContextGLX*>(stub_->decoder()->GetGLContext());
+#elif defined(OS_CHROMEOS) && defined(ARCH_CPU_X86_FAMILY)
   video_decode_accelerator_.reset(new VaapiVideoDecodeAccelerator(
-      glx_context->display(), make_context_current_));
-#elif defined(USE_OZONE)

[spang, 2014/08/29 18:51:27]

I'm concerned about removing this without providing an alternative. At least one
external project is using it.

We could maybe just do this for now:

#elif defined(OS_CHROMEOS) && defined(ARCH_CPU_X86_FAMILY)

// ChromeOS: use VA-API

#elif defined(USE_OZONE)

// !ChromeOS Ozone: Use MediaOzonePlatform

[llandwerlin-old, 2014/09/01 09:39:13]

Ok, I wasn't aware there was another project out there using the Ozone
abstraction. So far I didn't see any implementation of the Ozone interface and
it seems like an abstraction of this function.

[vignatti (out of this project), 2014/09/01 09:49:50]

yeah, Ozone-Wayland is using it at the moment:
https://github.com/01org/ozone-wayland/tree/master/media

and although I have not been active on the (internal) Ozone-GBM CL,  I was
planning to use the same interface for it:
https://codereview.chromium.org/385793002/

-  media::MediaOzonePlatform* platform =
-      media::MediaOzonePlatform::GetInstance();
-  video_decode_accelerator_.reset(platform->CreateVideoDecodeAccelerator(
-      make_context_current_));
-  if (!video_decode_accelerator_) {
-    SendCreateDecoderReply(init_done_msg, false);
-    return;
-  }
+      stub_->decoder()->GetGLContext(), make_context_current_));
 #elif defined(OS_ANDROID)
   video_decode_accelerator_.reset(new AndroidVideoDecodeAccelerator(
       stub_->decoder()->AsWeakPtr(),