Raster display item lists via a visual rect RTree.

cc/proto/display_item.proto

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 syntax = "proto2";
 
 import "skregion.proto";
 import "skrrect.proto";
 import "skxfermode.proto";
 import "rect.proto";
 import "rectf.proto";
 import "transform.proto";
 
 option optimize_for = LITE_RUNTIME;
 
 package cc.proto;
 
 message DisplayItemListSettings {
   optional bool use_cached_picture = 1;
 }
 
 message DisplayItemList {
   repeated DisplayItem items = 1;
+  // Deprecated.

[vmpstr, 2016/06/22 20:14:07]

I think at this point in time, we're free to remove things as need +nyquist@ for
an opinion

[nyquist, 2016/06/24 18:31:00]

I think that right now it's fine to remove this field. I'd still add the new
field with a new ID like you've done though.

[wkorman, 2016/06/24 19:57:14]

OK, removed. There are varying opinions on best practices to remove a proto
field even when in nascent usage, I've just literally deleted its entry here.
Some would say to leave it commented out for history, others would say to leave
it with a prefix like DELETED. Open to whatever is preferred.

[vmpstr, 2016/06/24 20:15:33]

I agree with deprecating fields in live code, since there would be clients in
existence that depend on the field. My recommendation here is only because this
is not launched yet, so we might as well do all the cleanup we can before it's
released. 

   optional cc.proto.Rect layer_rect = 2;
   optional DisplayItemListSettings settings = 3;
+  repeated cc.proto.Rect visual_rects = 4;
 }
 
 message DisplayItem {
   enum Type {
     Type_Clip = 1;
     Type_EndClip = 2;
     Type_ClipPath = 3;
     Type_EndClipPath = 4;
     Type_Compositing = 5;
     Type_EndCompositing = 6;
@@ skipping 48 matching lines @@
   // TODO(dtrainor): Support FilterOperations.
 }
 
 message FloatClipDisplayItem {
   optional cc.proto.RectF clip_rect = 1;
 }
 
 message TransformDisplayItem {
   optional cc.proto.Transform transform = 1;
 }