cc: Shrink size of display item

cc/playback/display_item.h

Index: cc/playback/display_item.h
diff --git a/cc/playback/display_item.h b/cc/playback/display_item.h
index e48d794682904419e9dddf37cab597694251cbe8..13c04b40e2abe3c06188d97684cc7bc0a4d38e3f 100644
--- a/cc/playback/display_item.h
+++ b/cc/playback/display_item.h
@@ -23,14 +23,6 @@ class CC_EXPORT DisplayItem {
  public:
   virtual ~DisplayItem() {}
 
-  void SetNew(bool is_suitable_for_gpu_rasterization,
-              int approximate_op_count,
-              size_t external_memory_usage) {
-    is_suitable_for_gpu_rasterization_ = is_suitable_for_gpu_rasterization;
-    approximate_op_count_ = approximate_op_count;
-    external_memory_usage_ = external_memory_usage;
-  }
-
   virtual void ToProtobuf(proto::DisplayItem* proto) const = 0;
   virtual void FromProtobuf(const proto::DisplayItem& proto) = 0;
   virtual void Raster(SkCanvas* canvas,
@@ -38,22 +30,10 @@ class CC_EXPORT DisplayItem {
                       SkPicture::AbortCallback* callback) const = 0;
   virtual void AsValueInto(const gfx::Rect& visual_rect,
                            base::trace_event::TracedValue* array) const = 0;
-
-  bool is_suitable_for_gpu_rasterization() const {
-    return is_suitable_for_gpu_rasterization_;
-  }
-  int approximate_op_count() const { return approximate_op_count_; }
-  size_t external_memory_usage() const { return external_memory_usage_; }
+  virtual size_t ExternalMemoryUsage() const = 0;

[danakj, 2015/12/08 19:18:14]

Can you leave a comment about this being for tracing, or maybe consider
ForTracing in the name?

[enne (OOO), 2015/12/08 19:47:27]

Left a comment.

 
  protected:
   DisplayItem();
-
-  bool is_suitable_for_gpu_rasterization_;
-  int approximate_op_count_;
-
-  // The size, in bytes, of the memory owned by this display item but not
-  // allocated within it (e.g. held through scoped_ptr or vector).
-  size_t external_memory_usage_;
 };
 
 }  // namespace cc