gfx: Make conversions from Size to SizeF be explicit.

media/blink/skcanvas_video_renderer.cc

Index: media/blink/skcanvas_video_renderer.cc
diff --git a/media/blink/skcanvas_video_renderer.cc b/media/blink/skcanvas_video_renderer.cc
index 3b64e38f81b30159eef8dc2d0fb4799ae29410f2..546ab8a358e636ff8ad79131ec0ede9546870e63 100644
--- a/media/blink/skcanvas_video_renderer.cc
+++ b/media/blink/skcanvas_video_renderer.cc
@@ -352,7 +352,7 @@ void SkCanvasVideoRenderer::Paint(const scoped_refptr<VideoFrame>& video_frame,
 
   const bool need_transform =
       video_rotation != VIDEO_ROTATION_0 ||
-      dest_rect.size() != video_frame->visible_rect().size() ||
+      dest_rect.size() != gfx::SizeF(video_frame->visible_rect().size()) ||

[no sievers, 2015/09/24 00:07:33]

hmm, do we really want to compare the rounded integer sizes here instead?

[danakj, 2015/09/24 00:36:05]

Ya I'm not sure, but this is what the code does today. Pointing out potential
wierdness is why I wanted to land this. I was going to +dalecurtis for media
afterward, but can do that now. :)

[DaleCurtis, 2015/09/24 01:40:09]

It seems like .size() should be dropped from both sides? I'd guess that's what
the original intention was here. I.e. use the implicit == which will compare the
integer width an height.

[danakj, 2015/09/24 17:48:35]

I'd like to not change behaviour in this patch, but we can do that in a
followup. That ok?

       !dest_rect.origin().IsOrigin();
   if (need_transform) {
     canvas->save();