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

services/ui/ws/gpu_service_proxy.h

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // 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;
 
   void CreateGpuMemoryBuffer(
       mojom::GpuMemoryBufferIdPtr id,
       const gfx::Size& size,
       gfx::BufferFormat format,
       gfx::BufferUsage usage,
       uint64_t surface_id,
       const mojom::GpuService::CreateGpuMemoryBufferCallback& callback)
       override;
 
   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_