cc : Fix damage rect bug caused by deletion of render surface

cc/trees/damage_tracker.h

Index: cc/trees/damage_tracker.h
diff --git a/cc/trees/damage_tracker.h b/cc/trees/damage_tracker.h
index d1ed0a1ac1f97b49fda7080fcc8b06fd395c8719..6d02af1ffe7c42b24b11cbb2c85fa718347fbe31 100644
--- a/cc/trees/damage_tracker.h
+++ b/cc/trees/damage_tracker.h
@@ -64,11 +64,15 @@ class CC_EXPORT DamageTracker {
                                     gfx::Rect* target_damage_rect);
 
   struct RectMapData {
-    RectMapData() : layer_id_(0), mailboxId_(0) {}
-    explicit RectMapData(int layer_id) : layer_id_(layer_id), mailboxId_(0) {}
-    void Update(const gfx::Rect& rect, unsigned int mailboxId) {
+    RectMapData() : layer_id_(0), mailboxId_(0), is_render_surface_(false) {}

[ajuma, 2016/05/16 15:16:21]

How about splitting the map in two, one map for layers and one for surfaces?
Then a change from surface to layer will be handled for free by the logic in
TrackDamageFromLeftoverRects (since we'll have a leftover rect in the surface
map). It also seems like something we'll need to do eventually anyway, to deal
with surfaces no longer being owned by layers.

[jaydasika, 2016/05/16 19:32:49]

Done.

+    explicit RectMapData(int layer_id)
+        : layer_id_(layer_id), mailboxId_(0), is_render_surface_(false) {}
+    void Update(const gfx::Rect& rect,
+                unsigned int mailboxId,
+                bool is_render_surface) {
       mailboxId_ = mailboxId;
       rect_ = rect;
+      is_render_surface_ = is_render_surface;
     }
 
     bool operator < (const RectMapData& other) const {
@@ -77,6 +81,7 @@ class CC_EXPORT DamageTracker {
 
     int layer_id_;
     unsigned int mailboxId_;
+    bool is_render_surface_;
     gfx::Rect rect_;
   };
   typedef std::vector<RectMapData> SortedRectMap;