Aggregate damage rects in surface aggregator

cc/surfaces/surface_aggregator.h

Index: cc/surfaces/surface_aggregator.h
diff --git a/cc/surfaces/surface_aggregator.h b/cc/surfaces/surface_aggregator.h
index 65174b531d89c0481b75b6eac2ecb569e14df1e9..a341bdeec134ffcc74fde41738ca8c70f0e9eced 100644
--- a/cc/surfaces/surface_aggregator.h
+++ b/cc/surfaces/surface_aggregator.h
@@ -29,9 +29,10 @@ class CC_SURFACES_EXPORT SurfaceAggregator {
   SurfaceAggregator(SurfaceManager* manager, ResourceProvider* provider);
   ~SurfaceAggregator();
 
-  scoped_ptr<CompositorFrame> Aggregate(
-      SurfaceId surface_id,
-      std::set<SurfaceId>* contained_surfaces);
+  scoped_ptr<CompositorFrame> Aggregate(SurfaceId surface_id);
+  base::hash_map<SurfaceId, int>& previous_contained_surfaces() {
+    return previous_contained_surfaces_;
+  }
 
  private:
   RenderPassId RemapPassId(RenderPassId surface_local_pass_id,
@@ -47,12 +48,15 @@ class CC_SURFACES_EXPORT SurfaceAggregator {
                        const gfx::Transform& content_to_target_transform,
                        RenderPass* dest_pass,
                        SurfaceId surface_id);
-  void CopyPasses(const RenderPassList& source_pass_list, SurfaceId surface_id);
+  void CopyPasses(const RenderPassList& source_pass_list,
+                  const Surface* surface);
 
   bool TakeResources(Surface* surface,
                      const DelegatedFrameData* frame_data,
                      RenderPassList* render_pass_list);
   int ChildIdForSurface(Surface* surface);
+  gfx::Rect DamageRectForSurface(const Surface* surface,
+                                 const RenderPass& source);
 
   SurfaceManager* manager_;
   ResourceProvider* provider_;
@@ -74,8 +78,10 @@ class CC_SURFACES_EXPORT SurfaceAggregator {
   typedef std::set<SurfaceId> SurfaceSet;
   SurfaceSet referenced_surfaces_;
 
-  // This is the set of surfaces that were used in the last draw.
-  SurfaceSet* contained_surfaces_;
+  // For each Surface used in the last aggregation, gives the frame_index at
+  // that time.
+  base::hash_map<SurfaceId, int> previous_contained_surfaces_;

[jamesr, 2014/08/26 22:06:22]

could use a typedef for this container type, it's used a lot

+  base::hash_map<SurfaceId, int> contained_surfaces_;
 
   // This is the pass list for the aggregated frame.
   RenderPassList* dest_pass_list_;