rendering_helper: Release GL context before terminate a display.

content/common/gpu/media/rendering_helper.cc

Index: content/common/gpu/media/rendering_helper.cc
diff --git a/content/common/gpu/media/rendering_helper.cc b/content/common/gpu/media/rendering_helper.cc
index 6abe8c5ca4dda93ac3032d70dee4deae488a1872..52ef543825576ee32b36e7449ac26ef120583181 100644
--- a/content/common/gpu/media/rendering_helper.cc
+++ b/content/common/gpu/media/rendering_helper.cc
@@ -157,7 +157,7 @@ void RenderingHelper::Initialize(const RenderingHelperParams& params,
 #endif
   gl_display_ = eglGetDisplay(native_display);
   CHECK(gl_display_);
-  CHECK(eglInitialize(gl_display_, NULL, NULL)) << glGetError();
+  CHECK(eglInitialize(gl_display_, NULL, NULL)) << eglGetError();
 
   static EGLint rgba8888[] = {
     EGL_RED_SIZE, 8,
@@ -398,6 +398,9 @@ void RenderingHelper::UnInitialize(base::WaitableEvent* done) {
 
   glXDestroyContext(x_display_, gl_context_);
 #else // EGL
+  CHECK(eglMakeCurrent(

[Pawel Osciak, 2014/06/19 03:59:23]

Don't we need to glxMakeCurrent for GLX above as well?

[Owen Lin, 2014/06/19 05:21:33]

No, it only needed for EGL. 
https://chromiumcodereview.appspot.com/271593010/diff/210001/content/common/g...

+      gl_display_, EGL_NO_SURFACE, EGL_NO_SURFACE, EGL_NO_CONTEXT))
+      << eglGetError();
   CHECK(eglDestroyContext(gl_display_, gl_context_));
   CHECK(eglDestroySurface(gl_display_, gl_surface_));
   CHECK(eglTerminate(gl_display_));