Remove ContentViewCore::GetDpiScale.

content/browser/android/overscroll_controller_android.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/android/overscroll_controller_android.h"
 
 #include "base/android/build_info.h"
 #include "base/command_line.h"
 #include "base/memory/ptr_util.h"
 #include "cc/layers/layer.h"
@@ skipping 79 matching lines @@
           switches::kDisablePullToRefreshEffect)) {
     return nullptr;
   }
 
   return base::WrapUnique(new OverscrollRefresh(handler));
 }
 
 }  // namespace
 
 OverscrollControllerAndroid::OverscrollControllerAndroid(
-    ContentViewCoreImpl* content_view_core)
+    ContentViewCoreImpl* content_view_core,

[no sievers, 2016/07/13 17:51:53]

Now you can pass OverscrollRefreshHandler* instead of CVC*.

[gsennton, 2016/07/13 18:38:39]

If we also pass the Compositor yup :P

[gsennton, 2016/07/13 19:24:30]

Can I submit as is or would you like me to change this as well?

[no sievers, 2016/07/13 19:26:32]

Ooops nevermind then.

Maybe eventually this can take a ViewAndroid or WindowAndroid.
And the OverscrollRefreshHandler (for pull-to-refresh) seems to be routed
through ContentViewCore.java for no particularly good reason. (Looks like we
could talk to SwipeRefreshHandler.java directly from here.) Filed
crbug.com/627943 for that.

+    float dpi_scale)
     : compositor_(content_view_core->GetWindowAndroid()->GetCompositor()),
-      dpi_scale_(content_view_core->GetDpiScale()),
+      dpi_scale_(dpi_scale),
       enabled_(true),
       glow_effect_(CreateGlowEffect(this, dpi_scale_)),
       refresh_effect_(CreateRefreshEffect(content_view_core)) {
   DCHECK(compositor_);
 }
 
 OverscrollControllerAndroid::~OverscrollControllerAndroid() {
 }
 
 bool OverscrollControllerAndroid::WillHandleGestureEvent(
@@ skipping 159 matching lines @@
 std::unique_ptr<EdgeEffectBase>
 OverscrollControllerAndroid::CreateEdgeEffect() {
   return CreateGlowEdgeEffect(&compositor_->GetResourceManager(), dpi_scale_);
 }
 
 void OverscrollControllerAndroid::SetNeedsAnimate() {
   compositor_->SetNeedsAnimate();
 }
 
 }  // namespace content