Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1301)

Unified Diff: content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java

Issue 2528823002: Separate SwipeRefreshHandler and ContentViewCore (Closed)
Patch Set: Remove unused SwipeRefreshHandler.mNativeSwipeRefreshHandler field Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java
diff --git a/content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java b/content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java
index 952f300d2e701811e9d720a2d1a3bea586a45b42..0a1339880cbe30130047e0df3cf0616d16800b3f 100644
--- a/content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java
+++ b/content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java
@@ -324,8 +324,6 @@ public class ContentViewCore implements AccessibilityStateChangeListener, Displa
private SelectPopup mSelectPopup;
private long mNativeSelectPopupSourceFrame = 0;
- private OverscrollRefreshHandler mOverscrollRefreshHandler;
-
private Runnable mFakeMouseMoveRunnable = null;
// Only valid when focused on a text / password field.
@@ -723,7 +721,6 @@ public class ContentViewCore implements AccessibilityStateChangeListener, Displa
try {
TraceEvent.begin("ContentViewCore.setContainerView");
if (mContainerView != null) {
- assert mOverscrollRefreshHandler == null;
hideSelectPopupWithCancelMessage();
mPopupZoomer.hide(false);
}
@@ -845,7 +842,6 @@ public class ContentViewCore implements AccessibilityStateChangeListener, Displa
// in this class.
mContentViewClient = new ContentViewClient();
mWebContents = null;
- mOverscrollRefreshHandler = null;
mNativeContentViewCore = 0;
mJavaScriptInterfaces.clear();
mRetainedJavaScriptObjects.clear();
@@ -2111,40 +2107,6 @@ public class ContentViewCore implements AccessibilityStateChangeListener, Displa
return new MotionEventSynthesizer(this);
}
- /**
- * Initialize the view with an overscroll refresh handler.
- * @param handler The refresh handler.
- */
- public void setOverscrollRefreshHandler(OverscrollRefreshHandler handler) {
- assert mOverscrollRefreshHandler == null || handler == null;
- mOverscrollRefreshHandler = handler;
- }
-
- @SuppressWarnings("unused")
- @CalledByNative
- private boolean onOverscrollRefreshStart() {
- if (mOverscrollRefreshHandler == null) return false;
- return mOverscrollRefreshHandler.start();
- }
-
- @SuppressWarnings("unused")
- @CalledByNative
- private void onOverscrollRefreshUpdate(float delta) {
- if (mOverscrollRefreshHandler != null) mOverscrollRefreshHandler.pull(delta);
- }
-
- @SuppressWarnings("unused")
- @CalledByNative
- private void onOverscrollRefreshRelease(boolean allowRefresh) {
- if (mOverscrollRefreshHandler != null) mOverscrollRefreshHandler.release(allowRefresh);
- }
-
- @SuppressWarnings("unused")
- @CalledByNative
- private void onOverscrollRefreshReset() {
- if (mOverscrollRefreshHandler != null) mOverscrollRefreshHandler.reset();
- }
-
@SuppressWarnings("unused")
@CalledByNative
private void onSelectionChanged(String text) {

Powered by Google App Engine
This is Rietveld 408576698