Update liftetime management of GpuClient

services/ui/ws/gpu_host.cc

 // 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.
 
 #include "services/ui/ws/gpu_host.h"
 
 #include "base/memory/ptr_util.h"
 #include "base/memory/shared_memory.h"
 #include "base/message_loop/message_loop.h"
 #include "base/run_loop.h"
@@ skipping 25 matching lines @@
 class GpuClient : public mojom::Gpu {
  public:
   GpuClient(int client_id,
             gpu::GPUInfo* gpu_info,
             ServerGpuMemoryBufferManager* gpu_memory_buffer_manager,
             mojom::GpuService* gpu_service)
       : client_id_(client_id),
         gpu_info_(gpu_info),
         gpu_memory_buffer_manager_(gpu_memory_buffer_manager),
         gpu_service_(gpu_service) {
+    LOG(ERROR) << "JR GpuClient device_id " << gpu_info->gpu.device_id << "\n";
     DCHECK(gpu_memory_buffer_manager_);
     DCHECK(gpu_service_);
   }
   ~GpuClient() override {
+    LOG(ERROR) << "JR GpuClient death\n";
     gpu_memory_buffer_manager_->DestroyAllGpuMemoryBufferForClient(client_id_);
   }
 
  private:
   void OnGpuChannelEstablished(const EstablishGpuChannelCallback& callback,
                                mojo::ScopedMessagePipeHandle channel_handle) {
+    LOG(ERROR) << "JR OnGpuChannelEstablished device_id "
+               << gpu_info_->gpu.device_id << "\n";
     callback.Run(client_id_, std::move(channel_handle), *gpu_info_);
   }
 
   // mojom::Gpu overrides:
   void EstablishGpuChannel(
       const EstablishGpuChannelCallback& callback) override {
     // TODO(sad): crbug.com/617415 figure out how to generate a meaningful
     // tracing id.
+    LOG(ERROR) << "JR GpuClient EstablishGpuChannel\n";
     const uint64_t client_tracing_id = 0;
     constexpr bool is_gpu_host = false;
     gpu_service_->EstablishGpuChannel(
         client_id_, client_tracing_id, is_gpu_host,
         base::Bind(&GpuClient::OnGpuChannelEstablished, base::Unretained(this),
                    callback));
   }
 
   void CreateGpuMemoryBuffer(
       gfx::GpuMemoryBufferId id,
@@ skipping 37 matching lines @@
   // TODO(sad): Correctly initialize gpu::GpuPreferences (like it is initialized
   // in GpuProcessHost::Init()).
   gpu::GpuPreferences preferences;
   gpu_main_->CreateGpuService(MakeRequest(&gpu_service_),
                               gpu_host_binding_.CreateInterfacePtrAndBind(),
                               preferences);
   gpu_memory_buffer_manager_ = base::MakeUnique<ServerGpuMemoryBufferManager>(
       gpu_service_.get(), next_client_id_++);
 }
 
-GpuHost::~GpuHost() {}
+GpuHost::~GpuHost() {
+  LOG(ERROR) << "JR GpuHost death\n";
+}
 
-void GpuHost::Add(mojom::GpuRequest request) {
-  mojo::MakeStrongBinding(
-      base::MakeUnique<GpuClient>(next_client_id_++, &gpu_info_,
-                                  gpu_memory_buffer_manager_.get(),
-                                  gpu_service_.get()),
-      std::move(request));
+mojom::Gpu* GpuHost::Add(mojom::GpuRequest request) {
+  //  gpu_bindings_.AddBinding(
+
+  return mojo::MakeStrongBinding(
+             base::MakeUnique<GpuClient>(next_client_id_++, &gpu_info_,
+                                         gpu_memory_buffer_manager_.get(),
+                                         gpu_service_.get()),
+             std::move(request))
+      ->impl();
 }
 
 void GpuHost::OnAcceleratedWidgetAvailable(gfx::AcceleratedWidget widget) {
 #if defined(OS_WIN)
   gfx::RenderingWindowManager::GetInstance()->RegisterParent(widget);
 #endif
 }
 
 void GpuHost::OnAcceleratedWidgetDestroyed(gfx::AcceleratedWidget widget) {
 #if defined(OS_WIN)
   gfx::RenderingWindowManager::GetInstance()->UnregisterParent(widget);
 #endif
 }
 
 void GpuHost::CreateDisplayCompositor(
     cc::mojom::DisplayCompositorRequest request,
     cc::mojom::DisplayCompositorClientPtr client) {
   gpu_main_->CreateDisplayCompositor(std::move(request), std::move(client));
 }
 
 void GpuHost::OnBadMessageFromGpu() {
   // TODO(sad): Received some unexpected message from the gpu process. We
   // should kill the process and restart it.
   NOTIMPLEMENTED();
 }
 
 void GpuHost::DidInitialize(const gpu::GPUInfo& gpu_info) {
+  LOG(ERROR) << "JR DidInitialize\n";
   gpu_info_ = gpu_info;
+  //  CHECK(false);
   delegate_->OnGpuServiceInitialized();
 }
 
 void GpuHost::DidCreateOffscreenContext(const GURL& url) {}
 
 void GpuHost::DidDestroyOffscreenContext(const GURL& url) {}
 
 void GpuHost::DidDestroyChannel(int32_t client_id) {}
 
 void GpuHost::DidLoseContext(bool offscreen,
@@ skipping 21 matching lines @@
   NOTREACHED();
 #endif
 }
 
 void GpuHost::StoreShaderToDisk(int32_t client_id,
                                 const std::string& key,
                                 const std::string& shader) {}
 
 }  // namespace ws
 }  // namespace ui