Remove VirtualGLApi

gpu/command_buffer/service/gl_context_virtual.cc

Index: gpu/command_buffer/service/gl_context_virtual.cc
diff --git a/gpu/command_buffer/service/gl_context_virtual.cc b/gpu/command_buffer/service/gl_context_virtual.cc
index 4c2776d4c8793c0d49372b952ce154f81682480f..6b763b6c55f1345d9da2178b373e6406ec5a0fee 100644
--- a/gpu/command_buffer/service/gl_context_virtual.cc
+++ b/gpu/command_buffer/service/gl_context_virtual.cc
@@ -25,22 +25,7 @@ GLContextVirtual::GLContextVirtual(gl::GLShareGroup* share_group,
 bool GLContextVirtual::Initialize(gl::GLSurface* compatible_surface,
                                   gl::GpuPreference gpu_preference) {
   SetGLStateRestorer(new GLStateRestorerImpl(decoder_));
-
-  // Virtual contexts obviously can't make a context that is compatible
-  // with the surface (the context already exists), but we do need to
-  // make a context current for SetupForVirtualization() below.
-  if (!IsCurrent(compatible_surface)) {
-    if (!shared_context_->MakeCurrent(compatible_surface)) {

[no sievers, 2016/07/08 17:52:19]

So this would potentially defer catching an incompatible surface until we really
have to make it current later, and probably only matter for onscreen.

I guess that's fine since it's more of a static assumption that we don't have
virtual context groups with incompatible surface configs.

[piman, 2016/07/08 18:30:58]

MakeVirtuallyCurrent below will effectively call MakeCurrent(shared_context_,
compatible_surface) - unless that combination is already current, so I think the
mismatch will be caught anyway.

It used to be that SetupForVirtualization needed the context to be current, but
since it's gone, I think this is not needed any more.

[no sievers, 2016/07/08 18:34:56]

Ah you're right, since this here was using the same permissive
GLContext*Virtual*::IsCurrent() check. Sounds good and definitely much simpler.

-      // This is likely an error. The real context should be made as
-      // compatible with all required surfaces when it was created.
-      LOG(ERROR) << "Failed MakeCurrent(compatible_surface)";
-      return false;
-    }
-  }
-
-  shared_context_->SetupForVirtualization();
-  shared_context_->MakeVirtuallyCurrent(this, compatible_surface);
-  return true;
+  return shared_context_->MakeVirtuallyCurrent(this, compatible_surface);
 }
 
 void GLContextVirtual::Destroy() {