Make lost context and error message callbacks on GpuControl go to client

mojo/gles2/gles2_context.cc

Index: mojo/gles2/gles2_context.cc
diff --git a/mojo/gles2/gles2_context.cc b/mojo/gles2/gles2_context.cc
index 90ba14e8c957a8d4d5edce7ca41098126a1d0f92..dad5f0c92d38f6fe1114820832d47d39b1d7e073 100644
--- a/mojo/gles2/gles2_context.cc
+++ b/mojo/gles2/gles2_context.cc
@@ -28,13 +28,14 @@ GLES2Context::GLES2Context(const std::vector<int32_t>& attribs,
                            mojo::ScopedMessagePipeHandle command_buffer_handle,
                            MojoGLES2ContextLost lost_callback,
                            void* closure)
-    : command_buffer_(this, attribs, std::move(command_buffer_handle)),
+    : command_buffer_(attribs, std::move(command_buffer_handle)),
       lost_callback_(lost_callback),
       closure_(closure) {}
 
 GLES2Context::~GLES2Context() {}
 
 bool GLES2Context::Initialize() {
+  command_buffer_.SetGpuControlClient(this);

[no sievers, 2016/04/07 21:07:58]

Won't GLES2Impl below clobber this and set itself as the client?

You probably want to set |lost_callback| as the callback on gles2 impl instead.

[danakj, 2016/04/07 21:45:19]

Yes I was doing this wrong like the GLInProcessContext before, fixed.

   if (!command_buffer_.Initialize())
     return false;
   gles2_helper_.reset(new gpu::gles2::GLES2CmdHelper(&command_buffer_));
@@ -63,6 +64,8 @@ bool GLES2Context::Initialize() {
                                      gpu::gles2::GLES2Implementation::kNoLimit);
 }
 
-void GLES2Context::ContextLost() { lost_callback_(closure_); }
+void GLES2Context::OnGpuControlLostContext() {
+  lost_callback_(closure_);
+}
 
 }  // namespace gles2