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..65cc1684dc37cf65d56ef16fdb123336a8baaacc 100644
--- a/cc/playback/display_item.h
+++ b/cc/playback/display_item.h
@@ -23,37 +23,19 @@ 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,
                       const gfx::Rect& canvas_target_playback_rect,
                       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_; }
+  // For tracing.
+  virtual size_t ExternalMemoryUsage() const = 0;
 
  protected:
-  DisplayItem();
-
-  bool is_suitable_for_gpu_rasterization_;
-  int approximate_op_count_;
+  virtual void FromProtobuf(const proto::DisplayItem& proto) = 0;

[danakj, 2015/12/08 19:52:04]

Why is this in the base class if it's protected? It seems like a helper method
called by the Proto constructors (should it just collapse into the
constructors?)

[enne (OOO), 2015/12/08 20:00:59]

Collapsed.  Removed.  Done.

 
-  // 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_;
+  DisplayItem();
 };
 
 }  // namespace cc