exo::CompositorFrameSinkHolder's release callbacks hold ref

components/exo/compositor_frame_sink_holder.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 COMPONENTS_EXO_COMPOSITOR_FRAME_SINK_HOLDER_H_
 #define COMPONENTS_EXO_COMPOSITOR_FRAME_SINK_HOLDER_H_
 
 #include <list>
 #include <map>
 #include <memory>
@@ skipping 61 matching lines @@
   // Overridden from SurfaceObserver:
   void OnSurfaceDestroying(Surface* surface) override;
 
  private:
   friend class base::RefCounted<CompositorFrameSinkHolder>;
 
   ~CompositorFrameSinkHolder() override;
 
   void UpdateNeedsBeginFrame();
 
-  // Each release callback holds a reference to the CompositorFrameSinkHolder
-  // itself to keep it alive. Running and erasing the callbacks might result in
-  // the instance being destroyed. Therefore, we should not access any member
-  // variables after running and erasing the callbacks.
+  // A collection of callbacks used to release resources.
   using ResourceReleaseCallbackMap =
-      std::map<int,
-               std::pair<scoped_refptr<CompositorFrameSinkHolder>,
-                         std::unique_ptr<cc::SingleReleaseCallback>>>;
+      std::map<int, std::unique_ptr<cc::SingleReleaseCallback>>;
   ResourceReleaseCallbackMap release_callbacks_;
 
   Surface* surface_;
   std::unique_ptr<CompositorFrameSink> frame_sink_;
 
   std::list<FrameCallback> active_frame_callbacks_;
   std::unique_ptr<cc::ExternalBeginFrameSource> begin_frame_source_;
   bool needs_begin_frame_ = false;
   cc::BeginFrameArgs last_begin_frame_args_;
   mojo::Binding<cc::mojom::MojoCompositorFrameSinkClient> binding_;
 
   base::WeakPtrFactory<CompositorFrameSinkHolder> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(CompositorFrameSinkHolder);
 };
 
 }  // namespace exo
 
 #endif  // COMPONENTS_EXO_COMPOSITOR_FRAME_SINK_HOLDER_H_