Merge "Prevent NPE in PulseExpansionHandler" into rvc-dev

This commit is contained in:
TreeHugger Robot
2020-05-21 01:57:11 +00:00
committed by Android (Google) Code Review

View File

@@ -121,14 +121,14 @@ constructor(
} }
override fun onInterceptTouchEvent(event: MotionEvent): Boolean { override fun onInterceptTouchEvent(event: MotionEvent): Boolean {
return maybeStartExpansion(event) return canHandleMotionEvent() && startExpansion(event)
} }
private fun maybeStartExpansion(event: MotionEvent): Boolean { private fun canHandleMotionEvent(): Boolean {
if (!wakeUpCoordinator.canShowPulsingHuns || qsExpanded || return wakeUpCoordinator.canShowPulsingHuns && !qsExpanded && !bouncerShowing
bouncerShowing) {
return false
} }
private fun startExpansion(event: MotionEvent): Boolean {
if (velocityTracker == null) { if (velocityTracker == null) {
velocityTracker = VelocityTracker.obtain() velocityTracker = VelocityTracker.obtain()
} }
@@ -177,8 +177,12 @@ constructor(
} }
override fun onTouchEvent(event: MotionEvent): Boolean { override fun onTouchEvent(event: MotionEvent): Boolean {
if (!isExpanding) { if (!canHandleMotionEvent()) {
return maybeStartExpansion(event) return false
}
if (!isExpanding || event.actionMasked == MotionEvent.ACTION_DOWN) {
return startExpansion(event)
} }
velocityTracker!!.addMovement(event) velocityTracker!!.addMovement(event)
val y = event.y val y = event.y