Return overlay resources to renderers in larger batches.

cc/resources/resource_provider.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_PROVIDER_H_
 #define CC_RESOURCES_RESOURCE_PROVIDER_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 506 matching lines @@
 #endif
 
   void WaitSyncTokenIfNeeded(ResourceId id);
 
   static GLint GetActiveTextureUnit(gpu::gles2::GLES2Interface* gl);
 
   void ValidateResource(ResourceId id) const;
 
   GLenum GetImageTextureTarget(gfx::BufferUsage usage, ResourceFormat format);
 
+  // Sets whether or not to store resources that are being returned to
+  // children to return them all at once. Setting this to false will return
+  // all accumulated resources.
+  void SetBatchReturnResources(bool aggregate);
+
   // base::trace_event::MemoryDumpProvider implementation.
   bool OnMemoryDump(const base::trace_event::MemoryDumpArgs& args,
                     base::trace_event::ProcessMemoryDump* pmd) override;
 
   int tracing_id() const { return tracing_id_; }
 
  private:
   struct Resource {
     enum Origin { INTERNAL, EXTERNAL, DELEGATED };
     enum SynchronizationState {
@@ skipping 230 matching lines @@
   BlockingTaskRunner* blocking_main_thread_task_runner_;
   bool lost_context_provider_;
   ResourceId next_id_;
   ResourceMap resources_;
   int next_child_;
   ChildMap children_;
   scoped_refptr<Fence> current_read_lock_fence_;
   std::unique_ptr<TextureIdAllocator> texture_id_allocator_;
   BufferToTextureTargetMap buffer_to_texture_target_map_;
 
+  // Keep track of whether deleted resources should be batched up or returned
+  // immediately.
+  bool batch_return_resources_ = false;
+  // Maps from a child id to the set of resources to be returned to it.
+  std::map<int, ResourceIdArray> batched_returning_resources_;

[Daniele Castagna, 2017/03/24 01:44:43]

How many child ids do you expect to have?
Should we use SmallMap instead?

+
   base::ThreadChecker thread_checker_;
 
   // A process-unique ID used for disambiguating memory dumps from different
   // resource providers.
   int tracing_id_;
 
 #if defined(OS_ANDROID)
   // Set of resource Ids that would like to be notified about promotion hints.
   ResourceIdSet wants_promotion_hints_set_;
 #endif
 
   DISALLOW_COPY_AND_ASSIGN(ResourceProvider);
 };
 
 }  // namespace cc
 
 #endif  // CC_RESOURCES_RESOURCE_PROVIDER_H_