Calculate damage rects before aggregating frame data.

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 d9466680113be22536aaa195fd05221a7e1d4c58..127a71b69db4a22177308a1228c58d14122ff510 100644
--- a/cc/surfaces/surface_aggregator.h
+++ b/cc/surfaces/surface_aggregator.h
@@ -72,14 +72,13 @@ class CC_SURFACES_EXPORT SurfaceAggregator {
       const ClipData& clip_rect,
       RenderPass* dest_pass,
       SurfaceId surface_id);
+  gfx::Rect ValidateAndCalculateDamageRect(SurfaceId surface_id);
   void CopyPasses(const DelegatedFrameData* frame_data, Surface* surface);
 
   // Remove Surfaces that were referenced before but aren't currently
   // referenced from the ResourceProvider.
   void RemoveUnreferencedChildren();
 
-  bool ValidateResources(Surface* surface,
-                         const DelegatedFrameData* frame_data);
   int ChildIdForSurface(Surface* surface);
   gfx::Rect DamageRectForSurface(const Surface* surface,
                                  const RenderPass& source,
@@ -111,6 +110,10 @@ class CC_SURFACES_EXPORT SurfaceAggregator {
   SurfaceIndexMap previous_contained_surfaces_;
   SurfaceIndexMap contained_surfaces_;
 
+  // For every Surface referenced in current frame is true if it's valid
+  // (resource references are consistent) and false if it isn't.
+  base::hash_map<SurfaceId, bool> surface_validity_;

[danakj, 2015/07/06 22:08:43]

how about a hash_set and you're valid if you're in it?

[jbauman, 2015/07/06 23:08:15]

Done.

+
   // This is the pass list for the aggregated frame.
   RenderPassList* dest_pass_list_;