Make content::BufferQueue use scoped ptrs for AllocatedSurfaces

content/browser/compositor/buffer_queue.h

 // Copyright 2014 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 CONTENT_BROWSER_COMPOSITOR_BUFFER_QUEUE_H_
 #define CONTENT_BROWSER_COMPOSITOR_BUFFER_QUEUE_H_
 
 #include <queue>
 #include <vector>
 
-#include "base/memory/linked_ptr.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
+#include "cc/base/scoped_ptr_deque.h"
+#include "cc/base/scoped_ptr_vector.h"
 #include "content/common/content_export.h"
 #include "ui/gfx/geometry/rect.h"
 #include "ui/gfx/geometry/size.h"
 
 namespace cc {
 class ContextProvider;
 }
 
 namespace gfx {
 class GpuMemoryBuffer;
@@ skipping 20 matching lines @@
 
   void Initialize();
 
   void BindFramebuffer();
   void SwapBuffers(const gfx::Rect& damage);
   void PageFlipComplete();
   void Reshape(const gfx::Size& size, float scale_factor);
 
   void RecreateBuffers();
 
-  unsigned int current_texture_id() const { return current_surface_.texture; }
+  unsigned int current_texture_id() const { return current_surface_->texture; }
   unsigned int fbo() const { return fbo_; }
 
  private:
   friend class BufferQueueTest;
 
   struct CONTENT_EXPORT AllocatedSurface {
     AllocatedSurface();
     AllocatedSurface(scoped_ptr<gfx::GpuMemoryBuffer> buffer,
                      unsigned int texture,
                      unsigned int image,
                      const gfx::Rect& rect);
     ~AllocatedSurface();
-    // TODO(ccameron): Change this to be a scoped_ptr, and change all use of
-    // AllocatedSurface to use scoped_ptr as well.
-    linked_ptr<gfx::GpuMemoryBuffer> buffer;
+    scoped_ptr<gfx::GpuMemoryBuffer> buffer;
     unsigned int texture;
     unsigned int image;
     gfx::Rect damage;  // This is the damage for this frame from the previous.
   };
 
   void FreeAllSurfaces();
 
-  void FreeSurface(AllocatedSurface* surface);
+  void FreeSurface(scoped_ptr<AllocatedSurface> surface);
 
   // Copy everything that is in |copy_rect|, except for what is in
   // |exclude_rect| from |source_texture| to |texture|.
   virtual void CopyBufferDamage(int texture,
                                 int source_texture,
                                 const gfx::Rect& new_damage,
                                 const gfx::Rect& old_damage);
 
   void UpdateBufferDamage(const gfx::Rect& damage);
 
   // Return a surface, available to be drawn into.
-  AllocatedSurface GetNextSurface();
+  scoped_ptr<AllocatedSurface> GetNextSurface();
 
-  AllocatedSurface RecreateBuffer(AllocatedSurface* surface);
+  scoped_ptr<AllocatedSurface> RecreateBuffer(
+      scoped_ptr<AllocatedSurface> surface);
 
   gfx::Size size_;
   scoped_refptr<cc::ContextProvider> context_provider_;
   unsigned int fbo_;
   size_t allocated_count_;
   unsigned int texture_target_;
   unsigned int internal_format_;
-  AllocatedSurface current_surface_;  // This surface is currently bound.
-  AllocatedSurface displayed_surface_;  // The surface currently on the screen.
-  std::vector<AllocatedSurface> available_surfaces_;  // These are free for use.
-  std::deque<AllocatedSurface> in_flight_surfaces_;
+  // This surface is currently bound.
+  scoped_ptr<AllocatedSurface> current_surface_;
+  // The surface currently on the screen.
+  scoped_ptr<AllocatedSurface> displayed_surface_;
+  // These are free for use.
+  cc::ScopedPtrVector<AllocatedSurface> available_surfaces_;

[reveman, 2015/10/27 17:03:44]

could we use base::ScopedVector instead?

+  cc::ScopedPtrDeque<AllocatedSurface> in_flight_surfaces_;
+  size_t deleted_in_flight_surface_count_;
   GLHelper* gl_helper_;
   BrowserGpuMemoryBufferManager* gpu_memory_buffer_manager_;
   int surface_id_;
 
   DISALLOW_COPY_AND_ASSIGN(BufferQueue);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_COMPOSITOR_BUFFER_QUEUE_H_