Move GpuMemoryBufferManager and SharedBitmapManager to CompositorFrameSink

ui/views/mus/surface_context_factory.cc

Index: ui/views/mus/surface_context_factory.cc
diff --git a/ui/views/mus/surface_context_factory.cc b/ui/views/mus/surface_context_factory.cc
index 626038b5b0568dbd3badf1b726e73f20054a4792..50ae575a3ea387bea6b0411721cf854d37426f94 100644
--- a/ui/views/mus/surface_context_factory.cc
+++ b/ui/views/mus/surface_context_factory.cc
@@ -41,9 +41,9 @@ void SurfaceContextFactory::CreateCompositorFrameSink(
   ui::mojom::CompositorFrameSinkType compositor_frame_sink_type =
       native_widget->compositor_frame_sink_type();
   auto compositor_frame_sink = window->RequestCompositorFrameSink(
-      compositor_frame_sink_type,
-      make_scoped_refptr(
-          new ui::ContextProvider(gpu_service_->EstablishGpuChannelSync())));
+      compositor_frame_sink_type, make_scoped_refptr(new ui::ContextProvider(
+                                      gpu_service_->EstablishGpuChannelSync())),
+      GetGpuMemoryBufferManager());

[danakj, 2016/10/27 21:00:34]

this could go to gpu_service_ right, do we need this in the API?

[piman, 2016/10/28 00:58:36]

I changed to gpu_service_->gpu_memory_buffer_manager() here (is that what you
were looking for?). GetGpuMemoryBufferManager needs to exist because of exo (see
above)

[danakj, 2016/10/28 01:33:20]

Was hoping the method could go away entirely but thanks for the explanation.

   compositor->SetCompositorFrameSink(std::move(compositor_frame_sink));
 }
 
@@ -78,11 +78,6 @@ uint32_t SurfaceContextFactory::GetImageTextureTarget(gfx::BufferFormat format,
   return GL_TEXTURE_2D;
 }
 
-cc::SharedBitmapManager* SurfaceContextFactory::GetSharedBitmapManager() {
-  // NOTIMPLEMENTED();
-  return nullptr;
-}
-
 gpu::GpuMemoryBufferManager*
 SurfaceContextFactory::GetGpuMemoryBufferManager() {
   return gpu_service_->gpu_memory_buffer_manager();