Remove the clip parameter from ContentLayerClient.

ui/compositor/layer.cc

Index: ui/compositor/layer.cc
diff --git a/ui/compositor/layer.cc b/ui/compositor/layer.cc
index 2ec6de99e156908a3a4d37dfa00ef353a7c4f2f9..aa46790f24f7994152cb0cc0c4fe04f630f11bf6 100644
--- a/ui/compositor/layer.cc
+++ b/ui/compositor/layer.cc
@@ -755,18 +755,16 @@ gfx::Rect Layer::PaintableRegion() {
 }
 
 scoped_refptr<cc::DisplayItemList> Layer::PaintContentsToDisplayList(
-    const gfx::Rect& clip,
     ContentLayerClient::PaintingControlSetting painting_control) {
   TRACE_EVENT1("ui", "Layer::PaintContentsToDisplayList", "name", name_);
   gfx::Rect local_bounds(bounds().size());
   gfx::Rect invalidation(
       gfx::IntersectRects(damaged_region_.bounds(), local_bounds));
-  DCHECK(clip.Contains(invalidation));
   ClearDamagedRects();
   cc::DisplayItemListSettings settings;
   settings.use_cached_picture = false;
   scoped_refptr<cc::DisplayItemList> display_list =
-      cc::DisplayItemList::Create(clip, settings);
+      cc::DisplayItemList::Create(PaintableRegion(), settings);

[enne (OOO), 2015/12/01 22:46:25]

Should this as well be the valid painted rect? Or, if it shouldn't be here,
should we remove this parameter?

[chrishtr, 2015/12/01 23:01:20]

Valid painted rect? Not sure what you mean.

Also, DisplayItemList has code to replay itself into an SkCanvas, which requires
bounds.

   if (delegate_) {
     delegate_->OnPaintLayer(
         PaintContext(display_list.get(), device_scale_factor_, invalidation));