Fix for 424682: Draw correct repaint regions instead of a conservative bounding box

cc/debug/debug_rect_history.cc

Index: cc/debug/debug_rect_history.cc
diff --git a/cc/debug/debug_rect_history.cc b/cc/debug/debug_rect_history.cc
index cde7765383f33ba59dccd77278663f9257d3976e..2c77760e907d5e0dccc34a1491089832ba71e17b 100644
--- a/cc/debug/debug_rect_history.cc
+++ b/cc/debug/debug_rect_history.cc
@@ -68,17 +68,24 @@ void DebugRectHistory::SavePaintRects(LayerImpl* layer) {
   // not. Therefore we traverse recursively over all layers, not just the render
   // surface list.
 
-  if (!layer->update_rect().IsEmpty() && layer->DrawsContent()) {
+  Region invalidation_region = layer->GetInvalidationRegion();
+  if (!invalidation_region.IsEmpty() && layer->DrawsContent()) {
     float width_scale = layer->content_bounds().width() /
                         static_cast<float>(layer->bounds().width());
     float height_scale = layer->content_bounds().height() /
                          static_cast<float>(layer->bounds().height());
-    gfx::Rect update_content_rect = gfx::ScaleToEnclosingRect(
-        layer->update_rect(), width_scale, height_scale);
-    debug_rects_.push_back(
+
+    for (Region::Iterator it(invalidation_region);
+        it.has_rect();

[danakj, 2015/03/02 20:10:53]

this doesn't look like how git cl format would do things, did you run it?
(PRESUBMIT warns you when you upload if you haven't)

[daplatz, 2015/03/02 20:28:02]

no i did not know about it. sweet.

+        it.next()) {
+

[danakj, 2015/03/02 20:10:53]

nit: remove this whitespace

[daplatz, 2015/03/02 20:28:02]

Acknowledged.

+      gfx::Rect update_content_rect = gfx::ScaleToEnclosingRect(
+        it.rect(), width_scale, height_scale);
+      debug_rects_.push_back(
         DebugRect(PAINT_RECT_TYPE,
-                  MathUtil::MapEnclosingClippedRect(
-                      layer->screen_space_transform(), update_content_rect)));
+          MathUtil::MapEnclosingClippedRect(
+            layer->screen_space_transform(), update_content_rect)));
+    }
   }
 
   for (unsigned i = 0; i < layer->children().size(); ++i)