cc: Partial tile update for one-copy raster.

cc/resources/resource_pool.h

 // Copyright 2012 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 CC_RESOURCES_RESOURCE_POOL_H_
 #define CC_RESOURCES_RESOURCE_POOL_H_
 
-#include <list>
+#include <deque>
 
 #include "base/memory/scoped_ptr.h"
 #include "cc/base/cc_export.h"
 #include "cc/output/renderer.h"
 #include "cc/resources/resource.h"
 #include "cc/resources/resource_format.h"
 
 namespace cc {
 class ScopedResource;
 
 class CC_EXPORT ResourcePool {
  public:
   static scoped_ptr<ResourcePool> Create(ResourceProvider* resource_provider,
                                          GLenum target) {
     return make_scoped_ptr(new ResourcePool(resource_provider, target));
   }
 
   virtual ~ResourcePool();
 
   scoped_ptr<ScopedResource> AcquireResource(const gfx::Size& size,
                                              ResourceFormat format);
-  void ReleaseResource(scoped_ptr<ScopedResource>);
+  scoped_ptr<ScopedResource> TryAcquireOldResource(const gfx::Size& size,

[reveman, 2015/05/22 17:15:13]

OldResource -> ResourceWithContentsId ?

[danakj, 2015/05/26 23:37:34]

Done.

+                                                   ResourceFormat format,
+                                                   uint64 id);

[reveman, 2015/05/22 17:15:13]

Looking at this class in isolation it's hard to understand what this "id" is.
There's also Resource->id() but that's a not the same but it's hard to see why
here. I think it would be easier to understand if this "id" here moved into the
Resource class as "contents ID" with an explicit description that it uniquely
identifies the contents.

[danakj, 2015/05/26 23:37:34]

Renamed to be explicitly content_id everywhere.

+  void ReleaseResource(scoped_ptr<ScopedResource> resource, uint64_t id);
 
   void SetResourceUsageLimits(size_t max_memory_usage_bytes,
                               size_t max_unused_memory_usage_bytes,
                               size_t max_resource_count);
 
   void ReduceResourceUsage();
   // This might block if |wait_if_needed| is true and one of the currently
   // busy resources has a read lock fence that needs to be waited upon before
   // it can be locked for write again.
   void CheckBusyResources(bool wait_if_needed);
 
   size_t total_memory_usage_bytes() const { return memory_usage_bytes_; }
   size_t acquired_memory_usage_bytes() const {
     return memory_usage_bytes_ - unused_memory_usage_bytes_;
   }
   size_t total_resource_count() const { return resource_count_; }
   size_t acquired_resource_count() const {
     return resource_count_ - unused_resources_.size();
   }
   size_t busy_resource_count() const { return busy_resources_.size(); }
 
  protected:
   ResourcePool(ResourceProvider* resource_provider, GLenum target);
 
   bool ResourceUsageTooHigh();
 
  private:
-  void DidFinishUsingResource(ScopedResource* resource);
+  void DidFinishUsingResource(ScopedResource* resource, uint64_t id);
 
   ResourceProvider* resource_provider_;
   const GLenum target_;
   size_t max_memory_usage_bytes_;
   size_t max_unused_memory_usage_bytes_;
   size_t max_resource_count_;
   size_t memory_usage_bytes_;
   size_t unused_memory_usage_bytes_;
   size_t resource_count_;
 
-  typedef std::list<ScopedResource*> ResourceList;
+  struct PoolResource {
+    PoolResource(ScopedResource* resource, uint64_t id)
+        : resource(resource), id(id) {}
+    ScopedResource* resource;
+    uint64_t id;
+  };
+  typedef std::deque<PoolResource> ResourceList;
   ResourceList unused_resources_;
   ResourceList busy_resources_;
 
   DISALLOW_COPY_AND_ASSIGN(ResourcePool);
 };
 
 }  // namespace cc
 
 #endif  // CC_RESOURCES_RESOURCE_POOL_H_