Remove ContentViewCore::GetDpiScale.

content/browser/android/overscroll_controller_android.cc

Index: content/browser/android/overscroll_controller_android.cc
diff --git a/content/browser/android/overscroll_controller_android.cc b/content/browser/android/overscroll_controller_android.cc
index 8feda1cba252ea5fbdf1d81fb68a07288ef163f4..5e8917259d7b39812766e11b054948d99a9c0622 100644
--- a/content/browser/android/overscroll_controller_android.cc
+++ b/content/browser/android/overscroll_controller_android.cc
@@ -97,9 +97,10 @@ std::unique_ptr<OverscrollRefresh> CreateRefreshEffect(
 }  // 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)) {