Mus: GpuCompositorFrameSink uses WeakPtrFactory

services/ui/surfaces/gpu_compositor_frame_sink.cc

 // Copyright 2015 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/surfaces/gpu_compositor_frame_sink.h"
 
 #include "base/callback.h"
 #include "base/message_loop/message_loop.h"
 #include "base/single_thread_task_runner.h"
 #include "base/threading/thread_task_runner_handle.h"
@@ skipping 22 matching lines @@
     scoped_refptr<cc::InProcessContextProvider> context_provider,
     cc::mojom::MojoCompositorFrameSinkRequest request,
     cc::mojom::MojoCompositorFrameSinkPrivateRequest private_request,
     cc::mojom::MojoCompositorFrameSinkClientPtr client)
     : frame_sink_id_(frame_sink_id),
       task_runner_(base::ThreadTaskRunnerHandle::Get()),
       display_compositor_(display_compositor),
       surface_factory_(frame_sink_id_, display_compositor_->manager(), this),
       client_(std::move(client)),
       binding_(this, std::move(request)),
-      private_binding_(this, std::move(private_request)) {
+      private_binding_(this, std::move(private_request)),
+      weak_factory_(this) {
   display_compositor_->manager()->RegisterFrameSinkId(frame_sink_id_);
   display_compositor_->manager()->RegisterSurfaceFactoryClient(frame_sink_id_,
                                                                this);
 
   if (surface_handle != gpu::kNullSurfaceHandle) {
     InitDisplay(surface_handle, gpu_memory_buffer_manager,
                 std::move(context_provider));
   }
 
   binding_.set_connection_error_handler(base::Bind(
@@ skipping 31 matching lines @@
     surface_factory_.Create(local_frame_id_);
     if (display_ && !frame.render_pass_list.empty()) {
       gfx::Size frame_size = frame.render_pass_list[0]->output_rect.size();
       display_->Resize(frame_size);
     }
   }
   ++ack_pending_count_;
   surface_factory_.SubmitCompositorFrame(
       local_frame_id_, std::move(frame),
       base::Bind(&GpuCompositorFrameSink::DidReceiveCompositorFrameAck,
-                 base::Unretained(this)));
+                 weak_factory_.GetWeakPtr()));
   if (display_) {
     display_->SetLocalFrameId(local_frame_id_,
                               frame.metadata.device_scale_factor);
   }
 }
 
 void GpuCompositorFrameSink::DidReceiveCompositorFrameAck() {
   if (!client_)
     return;
   client_->DidReceiveCompositorFrameAck();
@@ skipping 132 matching lines @@
 }
 
 void GpuCompositorFrameSink::OnPrivateConnectionLost() {
   private_connection_lost_ = true;
   // Request destruction of |this| only if both connections are lost.
   display_compositor_->OnCompositorFrameSinkPrivateConnectionLost(
       frame_sink_id_, client_connection_lost_);
 }
 
 }  // namespace ui