Getting rid of frame_size_in_dip in Layer::SetShowSurface

content/browser/renderer_host/delegated_frame_host.cc

Index: content/browser/renderer_host/delegated_frame_host.cc
diff --git a/content/browser/renderer_host/delegated_frame_host.cc b/content/browser/renderer_host/delegated_frame_host.cc
index 0ba9c7bd0fa57e2bd642aa7ba38994637dbb24c6..c4d799ebfd99c2483f7c5b4f85750d0a54e1b2c0 100644
--- a/content/browser/renderer_host/delegated_frame_host.cc
+++ b/content/browser/renderer_host/delegated_frame_host.cc
@@ -514,7 +514,7 @@ void DelegatedFrameHost::SwapDelegatedFrame(uint32_t compositor_frame_sink_id,
           cc::SurfaceId(frame_sink_id_, local_frame_id_),
           base::Bind(&SatisfyCallback, base::Unretained(manager)),
           base::Bind(&RequireCallback, base::Unretained(manager)), frame_size,
-          frame_device_scale_factor, frame_size_in_dip);

[Fady Samuel, 2016/12/14 00:33:16]

Do we still need frame_size_in_dip? Can you delete that?

[Saman Sami, 2016/12/14 00:39:46]

It's used in a few if conditions above and then it's assigned to
current_frame_size_in_dip_ below

+          frame_device_scale_factor);
       current_surface_size_ = frame_size;
       current_scale_factor_ = frame_device_scale_factor;
     }