Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

DO NOT MERGE - Make WebView track focal points more accurately

Browse Source 
Change-Id: I6683edee6933b1dd026453935d72119320ab3c81
This commit is contained in:
Adam Powell
2012-08-31 17:19:24 -07:00
parent c4ce0da255
commit 9f3b0bfd0b
2 changed files with 31 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

43
core/java/android/webkit/WebViewClassic.java
View File

@@ -1336,20 +1336,40 @@ public final class WebViewClassic implements WebViewProvider, WebViewProvider.Sc
private void onHandleUiTouchEvent(MotionEvent ev) {
final ScaleGestureDetector detector =
mZoomManager.getMultiTouchGestureDetector();
mZoomManager.getScaleGestureDetector();
float x = ev.getX();
float y = ev.getY();
int action = ev.getActionMasked();
final boolean pointerUp = action == MotionEvent.ACTION_POINTER_UP;
final boolean configChanged =
action == MotionEvent.ACTION_POINTER_UP ||
action == MotionEvent.ACTION_POINTER_DOWN;
final int skipIndex = pointerUp ? ev.getActionIndex() : -1;
// Determine focal point
float sumX = 0, sumY = 0;
final int count = ev.getPointerCount();
for (int i = 0; i < count; i++) {
if (skipIndex == i) continue;
sumX += ev.getX(i);
sumY += ev.getY(i);
}
final int div = pointerUp ? count - 1 : count;
float x = sumX / div;
float y = sumY / div;
if (configChanged) {
mLastTouchX = Math.round(x);
mLastTouchY = Math.round(y);
mLastTouchTime = ev.getEventTime();
mWebView.cancelLongPress();
mPrivateHandler.removeMessages(SWITCH_TO_LONGPRESS);
}
if (detector != null) {
detector.onTouchEvent(ev);
if (detector.isInProgress()) {
mLastTouchTime = ev.getEventTime();
x = detector.getFocusX();
y = detector.getFocusY();
mWebView.cancelLongPress();
mPrivateHandler.removeMessages(SWITCH_TO_LONGPRESS);
if (!mZoomManager.supportsPanDuringZoom()) {
return;
}
@@ -1360,14 +1380,9 @@ public final class WebViewClassic implements WebViewProvider, WebViewProvider.Sc
}
}
int action = ev.getActionMasked();
if (action == MotionEvent.ACTION_POINTER_DOWN) {
cancelTouch();
action = MotionEvent.ACTION_DOWN;
} else if (action == MotionEvent.ACTION_POINTER_UP) {
// set mLastTouchX/Y to the remaining points for multi-touch.
mLastTouchX = Math.round(x);
mLastTouchY = Math.round(y);
} else if (action == MotionEvent.ACTION_MOVE) {
// negative x or y indicate it is on the edge, skip it.
if (x < 0 || y < 0) {
@@ -4345,7 +4360,7 @@ public final class WebViewClassic implements WebViewProvider, WebViewProvider.Sc
// A multi-finger gesture can look like a long press; make sure we don't take
// long press actions if we're scaling.
final ScaleGestureDetector detector = mZoomManager.getMultiTouchGestureDetector();
final ScaleGestureDetector detector = mZoomManager.getScaleGestureDetector();
if (detector != null && detector.isInProgress()) {
return false;
}
@@ -5752,7 +5767,7 @@ public final class WebViewClassic implements WebViewProvider, WebViewProvider.Sc
* and the middle point for multi-touch.
*/
private void handleTouchEventCommon(MotionEvent event, int action, int x, int y) {
ScaleGestureDetector detector = mZoomManager.getMultiTouchGestureDetector();
ScaleGestureDetector detector = mZoomManager.getScaleGestureDetector();
long eventTime = event.getEventTime();

4
core/java/android/webkit/ZoomManager.java
View File

@@ -204,7 +204,7 @@ class ZoomManager {
*/
private boolean mAllowPanAndScale;
// use the framework's ScaleGestureDetector to handle multi-touch
// use the framework's ScaleGestureDetector to handle scaling gestures
private ScaleGestureDetector mScaleDetector;
private boolean mPinchToZoomAnimating = false;
@@ -768,7 +768,7 @@ class ZoomManager {
return isZoomAnimating();
}
public ScaleGestureDetector getMultiTouchGestureDetector() {
public ScaleGestureDetector getScaleGestureDetector() {
return mScaleDetector;
}