services/ui: Rename a few gpu-related classes.

services/ui/ws/gpu_service_proxy.h

Index: services/ui/ws/gpu_service_proxy.h
diff --git a/services/ui/gpu/gpu_service_impl.h b/services/ui/ws/gpu_service_proxy.h
similarity index 60%
rename from services/ui/gpu/gpu_service_impl.h
rename to services/ui/ws/gpu_service_proxy.h
index 677923e8562b077c1c83849f266159f370b731dd..1650c523d92f363664864f0a76a6ffc2d88e42c2 100644
--- a/services/ui/gpu/gpu_service_impl.h
+++ b/services/ui/ws/gpu_service_proxy.h
@@ -2,21 +2,28 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#ifndef SERVICES_UI_GPU_GPU_SERVICE_IMPL_H_
-#define SERVICES_UI_GPU_GPU_SERVICE_IMPL_H_
+#ifndef SERVICES_UI_WS_GPU_SERVICE_PROXY_H_
+#define SERVICES_UI_WS_GPU_SERVICE_PROXY_H_
 
+#include "mojo/public/cpp/bindings/binding_set.h"
 #include "mojo/public/cpp/bindings/interface_request.h"
-#include "mojo/public/cpp/bindings/strong_binding.h"
 #include "services/ui/public/interfaces/gpu_memory_buffer.mojom.h"
 #include "services/ui/public/interfaces/gpu_service.mojom.h"
 
 namespace ui {
 
-class GpuServiceImpl : public mojom::GpuService {
+class GpuServiceInternal;
+
+// The proxy implementation that relays requests from clients to the real
+// service implementation in the GPU process over mojom.GpuServiceInternal.
+class GpuServiceProxy : public mojom::GpuService {

[Fady Samuel, 2016/08/24 15:44:11]

Hmm, do we know that the gpu service interface will be identical to this "proxy"
or should we have two interfaces?

[sadrul, 2016/08/24 15:47:53]

This is the client-facing interface. The one between ws and gpu
(GpuServiceInternal) will have a slightly different API (e.g. it will include
the Initialize()/Crash() etc. things too, and the signatures will also be a bit
different).

  public:
-  GpuServiceImpl(mojo::InterfaceRequest<mojom::GpuService> request);
-  ~GpuServiceImpl() override;
+  GpuServiceProxy();
+  ~GpuServiceProxy() override;
+
+  void Add(mojom::GpuServiceRequest request);
 
+ private:
   // mojom::GpuService overrides:
   void EstablishGpuChannel(
       const mojom::GpuService::EstablishGpuChannelCallback& callback) override;
@@ -33,12 +40,12 @@ class GpuServiceImpl : public mojom::GpuService {
   void DestroyGpuMemoryBuffer(mojom::GpuMemoryBufferIdPtr id,
                               const gpu::SyncToken& sync_token) override;
 
- private:
-  mojo::StrongBinding<GpuService> binding_;
+  GpuServiceInternal* gpu_service_;
+  mojo::BindingSet<GpuService> bindings_;
 
-  DISALLOW_COPY_AND_ASSIGN(GpuServiceImpl);
+  DISALLOW_COPY_AND_ASSIGN(GpuServiceProxy);
 };
 
 }  // namespace ui
 
-#endif  // SERVICES_UI_GPU_GPU_SERVICE_IMPL_H_
+#endif  // SERVICES_UI_WS_GPU_SERVICE_PROXY_H_