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

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

Issue 23533051: [android_webview] Use a fraction to calculate scroll offset. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: revert changes from previous patch set as they break AwSettings tests Created 7 years, 3 months 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 88785c177e4d7083df2badb0f07f756141072d1b..1ac3e5c26bfe2d53787016ad32768e992a37b9ef 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
@@ -244,11 +244,9 @@ import java.util.Map;
/**
* Called each time any of the parameters are changed.
*
- * @param widthCss The content width in logical (CSS) pixels.
- * @param heightCss The content height in logical (CSS) pixels.
* @param pageScaleFactor The page scale.
*/
- void onFrameInfoUpdated(float widthCss, float heightCss, float pageScaleFactor);
+ void onFrameInfoUpdated(float pageScaleFactor);
}
private VSyncManager.Provider mVSyncProvider;
@@ -359,7 +357,6 @@ import java.util.Map;
private ContentViewGestureHandler mContentViewGestureHandler;
private GestureStateListener mGestureStateListener;
- private UpdateFrameInfoListener mUpdateFrameInfoListener;
private ZoomManager mZoomManager;
private ZoomControlsDelegate mZoomControlsDelegate;
@@ -2337,10 +2334,6 @@ import java.util.Map;
if (mNativeContentViewCore != 0) nativeShowImeIfNeeded(mNativeContentViewCore);
}
- public void setUpdateFrameInfoListener(UpdateFrameInfoListener updateFrameInfoListener) {
- mUpdateFrameInfoListener = updateFrameInfoListener;
- }
-
@SuppressWarnings("unused")
@CalledByNative
private void updateFrameInfo(
@@ -2396,11 +2389,6 @@ import java.util.Map;
pageScaleFactor, minPageScaleFactor, maxPageScaleFactor,
contentOffsetYPix);
- if ((contentSizeChanged || pageScaleChanged) && mUpdateFrameInfoListener != null) {
- mUpdateFrameInfoListener.onFrameInfoUpdated(
- contentWidth, contentHeight, pageScaleFactor);
- }
-
if (needTemporarilyHideHandles) temporarilyHideTextHandles();
if (needUpdateZoomControls) mZoomControlsDelegate.updateZoomControls();
if (contentOffsetChanged) updateHandleScreenPositions();

Powered by Google App Engine
This is Rietveld 408576698