Separate SwipeRefreshHandler and ContentViewCore

ui/android/overscroll_refresh_unittest.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 "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 {
     started_ = true;
     return true;
   }
 
   void PullUpdate(float delta) override { delta_ += delta; }
 
   void PullRelease(bool allow_refresh) override {
@@ skipping 207 matching lines @@
   EXPECT_TRUE(GetAndResetPullStarted());
 
   // An early reset should prevent the refresh action from firing.
   effect.Reset();
   EXPECT_TRUE(GetAndResetPullReset());
   effect.OnScrollEnd(gfx::Vector2dF());
   EXPECT_FALSE(GetAndResetPullReleased());
 }
 
 }  // namespace ui