Impl mus::mojom::GpuService to enable using Chrome IPC version gpu CmdBuf in mus

ui/views/mus/surface_binding.cc

Index: ui/views/mus/surface_binding.cc
diff --git a/ui/views/mus/surface_binding.cc b/ui/views/mus/surface_binding.cc
index 415e115535cdc3714697f28e58b12dcb913465bb..454ef4920daea85f12137ffa7633fd12c474b100 100644
--- a/ui/views/mus/surface_binding.cc
+++ b/ui/views/mus/surface_binding.cc
@@ -24,9 +24,7 @@
 #include "components/mus/public/cpp/surfaces/surfaces_type_converters.h"
 #include "components/mus/public/cpp/window.h"
 #include "components/mus/public/cpp/window_tree_client.h"
-#include "components/mus/public/interfaces/gpu.mojom.h"
 #include "mojo/public/cpp/bindings/binding.h"
-#include "services/shell/public/cpp/connector.h"
 #include "ui/views/mus/window_tree_host_mus.h"
 
 namespace views {
@@ -56,17 +54,12 @@ class SurfaceBinding::PerClientState : public base::RefCounted<PerClientState> {
                  mus::WindowTreeClient* client);
   ~PerClientState();
 
-  void Init();
-
   static base::LazyInstance<
       base::ThreadLocalPointer<ClientToStateMap>>::Leaky window_states;
 
   shell::Connector* connector_;
   mus::WindowTreeClient* client_;
 
-  // Set of state needed to create an OutputSurface.
-  mus::mojom::GpuPtr gpu_;
-
   DISALLOW_COPY_AND_ASSIGN(PerClientState);
 };
 
@@ -84,10 +77,8 @@ SurfaceBinding::PerClientState* SurfaceBinding::PerClientState::Get(
     window_map = new ClientToStateMap;
     window_states.Pointer()->Set(window_map);
   }
-  if (!(*window_map)[client]) {
+  if (!(*window_map)[client])
     (*window_map)[client] = new PerClientState(connector, client);
-    (*window_map)[client]->Init();
-  }
   return (*window_map)[client];
 }
 
@@ -95,15 +86,8 @@ std::unique_ptr<cc::OutputSurface>
 SurfaceBinding::PerClientState::CreateOutputSurface(
     mus::Window* window,
     mus::mojom::SurfaceType surface_type) {
-  if (gpu_.encountered_error())
-    return nullptr;
-  // TODO(sky): figure out lifetime here. Do I need to worry about the return
-  // value outliving this?
-  mus::mojom::CommandBufferPtr cb;
-  gpu_->CreateOffscreenGLES2Context(GetProxy(&cb));
-
   scoped_refptr<cc::ContextProvider> context_provider(
-      new mus::ContextProvider(cb.PassInterface().PassHandle()));
+      new mus::ContextProvider(connector_));
   return base::WrapUnique(new mus::OutputSurface(
       context_provider, window->RequestSurface(surface_type)));
 }
@@ -124,16 +108,6 @@ SurfaceBinding::PerClientState::~PerClientState() {
   }
 }
 
-void SurfaceBinding::PerClientState::Init() {
-  connector_->ConnectToInterface("mojo:mus", &gpu_);
-
-  // TODO(sad): If service connection is lost (e.g. if gpu crashes), then the
-  // clients need to be restored. https://crbug.com/613366
-  // TODO(rockot|yzshen): It is necessary to install a connection-error handler,
-  // even if the handler doesn't actually do anything. https://crbug.com/613371
-  gpu_.set_connection_error_handler([]{});

[sky, 2016/06/01 15:45:29]

Where does this code end up now?

[Peng, 2016/06/01 16:23:21]

With CL, we will create a new GpuPtr in GLES2Context::Initialize() every time,
and that method will find out errors if GPU crashes. So I think we don't need
check the error here anymore.

-}
-
 // SurfaceBinding --------------------------------------------------------------
 
 SurfaceBinding::SurfaceBinding(shell::Connector* connector,