Separate SwipeRefreshHandler and ContentViewCore

ui/android/overscroll_refresh_unittest.cc

Index: ui/android/overscroll_refresh_unittest.cc
diff --git a/ui/android/overscroll_refresh_unittest.cc b/ui/android/overscroll_refresh_unittest.cc
index 39a8b308a33a3ee26c6dde2a9d36ca618530e45c..9f399772d44e79b64ba775c74f072b7485d41da7 100644
--- a/ui/android/overscroll_refresh_unittest.cc
+++ b/ui/android/overscroll_refresh_unittest.cc
@@ -2,16 +2,19 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
+#include "base/android/scoped_java_ref.h"
 #include "cc/layers/layer.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/android/overscroll_refresh.h"
+#include "ui/android/overscroll_refresh_handler.h"
 
 namespace ui {
 
 class OverscrollRefreshTest : public OverscrollRefreshHandler,
                               public testing::Test {
  public:
-  OverscrollRefreshTest() {}
+  OverscrollRefreshTest()
+      : OverscrollRefreshHandler(base::android::JavaRef<jobject>()) {}

[boliu, 2016/12/06 16:06:45]

nit: I *think* nullptr is good enough?

[rlanday, 2016/12/06 19:01:16]

Ok I'll try that

 
   // OverscrollRefreshHandler implementation.
   bool PullStart() override {