Partial swap implementation for surfaceless

content/browser/compositor/gpu_process_transport_factory.cc

Index: content/browser/compositor/gpu_process_transport_factory.cc
diff --git a/content/browser/compositor/gpu_process_transport_factory.cc b/content/browser/compositor/gpu_process_transport_factory.cc
index 766bb3e7438a7a8eb7f339c991b259e05df133c5..39b5d8eb921c08482961a4964e2391acc7097425 100644
--- a/content/browser/compositor/gpu_process_transport_factory.cc
+++ b/content/browser/compositor/gpu_process_transport_factory.cc
@@ -240,6 +240,7 @@ scoped_ptr<cc::OutputSurface> GpuProcessTransportFactory::CreateOutputSurface(
         &output_surface_map_,
         compositor->vsync_manager(),
         CreateOverlayCandidateValidator(compositor->widget()),
+        GetGLHelper(),

[alexst (slow to review), 2014/09/17 20:34:06]

I don't think this is safe to do. If a context is lost, the helper will get
reset, so you'll be storing a deleted object.

GpuProcessTransportFactory implements ImageTransportFactory, which has a
GetInstance and a getter for GLHelper. We should get the helper directly from
there before each use.

[achaulk, 2014/09/17 20:41:57]

Done.

         GL_RGB8_OES));
   }
 #endif