Getting rid of frame_size_in_dip in Layer::SetShowSurface

ui/compositor/layer.h

Index: ui/compositor/layer.h
diff --git a/ui/compositor/layer.h b/ui/compositor/layer.h
index 4bb77d315bdc7e96014a88a6b7a7e7e874c27ae5..66fa208398263a6a1702ab7c8b0b5321967b9cf7 100644
--- a/ui/compositor/layer.h
+++ b/ui/compositor/layer.h
@@ -297,8 +297,7 @@ class COMPOSITOR_EXPORT Layer
                       const cc::SurfaceLayer::SatisfyCallback& satisfy_callback,
                       const cc::SurfaceLayer::RequireCallback& require_callback,
                       gfx::Size surface_size,

[sadrul, 2016/12/14 01:06:25]

Can you rename this to be |surface_size_in_pixels| to make it obvious that this
is not in DIP?

[Fady Samuel, 2016/12/14 01:11:44]

While you're at it, we tend to prefer pass by const ref for non-primitive types.

const gfx::Size& surface_size_in_pixels

[Saman Sami, 2016/12/14 01:28:02]

Done.

-                      float scale,
-                      gfx::Size frame_size_in_dip);
+                      float scale);
 
   bool has_external_content() {
     return texture_layer_.get() || surface_layer_.get();