don't use deprecated helper methods on SkCanvas

cc/layers/picture_image_layer_unittest.cc

Index: cc/layers/picture_image_layer_unittest.cc
diff --git a/cc/layers/picture_image_layer_unittest.cc b/cc/layers/picture_image_layer_unittest.cc
index 2280494bd4c33e5cf0ad6254860512e965b6dcdf..ecdf15bbf0aa99fa486ddd4d0978148e09f0101c 100644
--- a/cc/layers/picture_image_layer_unittest.cc
+++ b/cc/layers/picture_image_layer_unittest.cc
@@ -37,8 +37,8 @@ TEST(PictureImageLayerTest, PaintContentsToDisplayList) {
   image_canvas->clear(SK_ColorRED);
   SkPaint blue_paint;
   blue_paint.setColor(SK_ColorBLUE);
-  image_canvas->drawRectCoords(0.f, 0.f, 100.f, 100.f, blue_paint);
-  image_canvas->drawRectCoords(100.f, 100.f, 200.f, 200.f, blue_paint);
+  image_canvas->drawRect({0.f, 0.f, 100.f, 100.f}, blue_paint);

[f(malita), 2017/02/24 21:37:19]

I find initializer lists fragile.  Is that LTRB or XYWH?  What if the internal
structure of SkRect changes?

MakeWH/MakeLTRB/etc. would be a clearer choice, no?

[reed1, 2017/02/24 21:39:02]

We are using these internally now as well. There is no way we could change it.

+  image_canvas->drawRect({100.f, 100.f, 200.f, 200.f}, blue_paint);
 
   layer->SetImage(image_surface->makeImageSnapshot());
   layer->SetBounds(gfx::Size(layer_rect.width(), layer_rect.height()));