Constify SkPicture in DrawingDisplayItem (cc side)

cc/blink/web_display_item_list_impl.h

Index: cc/blink/web_display_item_list_impl.h
diff --git a/cc/blink/web_display_item_list_impl.h b/cc/blink/web_display_item_list_impl.h
index cd35975caeb5ae531bf631108b993ab6a674aa41..dee4ad452b516d542c7855c3da485ff2c5b84eff 100644
--- a/cc/blink/web_display_item_list_impl.h
+++ b/cc/blink/web_display_item_list_impl.h
@@ -14,6 +14,7 @@
 #include "third_party/WebKit/public/platform/WebFloatPoint.h"
 #include "third_party/WebKit/public/platform/WebVector.h"
 #include "third_party/skia/include/core/SkRegion.h"
+#include "ui/gfx/geometry/point_f.h"
 
 class SkImageFilter;
 class SkMatrix44;
@@ -31,8 +32,12 @@ class WebDisplayItemListImpl : public blink::WebDisplayItemList {
   scoped_refptr<cc::DisplayItemList> ToDisplayItemList();
 
   // blink::WebDisplayItemList implementation.
-  virtual void appendDrawingItem(SkPicture* picture,
-                                 const blink::WebFloatPoint& location);
+  virtual void appendDrawingItem(
+      const SkPicture*,
+      const blink::WebFloatPoint& location = gfx::PointF());
+  virtual void appendDrawingItem(
+      SkPicture*,
+      const blink::WebFloatPoint& location = gfx::PointF());

[ajuma, 2015/02/12 19:42:59]

I think we can do without the default arguments here, since Blink is currently
calling only the second version with both arguments, and will soon only call the
first version with one argument.

[Stephen Chennney, 2015/02/12 19:47:59]

Yes, I'll verify.

   virtual void appendClipItem(
       const blink::WebRect& clip_rect,
       const blink::WebVector<SkRRect>& rounded_clip_rects);