fix two crashes
* fix crash where Workspace gets an ACTION_MOVE but didn't get the ACTION_DOWN before it * fix crash in DimmableBubbleTextView on non-xlarge devices (buganizer bug#3001211) Change-Id: I419b4a9fd3c95df9949f1d46746845ad2dc4b831
This commit is contained in:
@@ -545,15 +545,21 @@ public abstract class PagedView extends ViewGroup {
|
||||
return true;
|
||||
}
|
||||
|
||||
|
||||
switch (action & MotionEvent.ACTION_MASK) {
|
||||
case MotionEvent.ACTION_MOVE: {
|
||||
/*
|
||||
* mIsBeingDragged == false, otherwise the shortcut would have caught it. Check
|
||||
* whether the user has moved far enough from his original down touch.
|
||||
*/
|
||||
determineScrollingStart(ev);
|
||||
break;
|
||||
if (mActivePointerId != INVALID_POINTER) {
|
||||
determineScrollingStart(ev);
|
||||
break;
|
||||
}
|
||||
// if mActivePointerId is INVALID_POINTER, then we must have missed an ACTION_DOWN
|
||||
// event. in that case, treat the first occurence of a move event as a ACTION_DOWN
|
||||
// i.e. fall through to the next case (don't break)
|
||||
// (We sometimes miss ACTION_DOWN events in Workspace because it ignores all events
|
||||
// while it's small- this was causing a crash before we checked for INVALID_POINTER)
|
||||
}
|
||||
|
||||
case MotionEvent.ACTION_DOWN: {
|
||||
|
||||
Reference in New Issue
Block a user