feat(android): fire touch events synchronously

android/modules/ui/src/java/ti/modules/titanium/ui/widget/TiUIButton.java

@@ -51,7 +51,7 @@ public boolean onFilterTouchEventForSecurity(MotionEvent event)
 			{
 				boolean isTouchAllowed = super.onFilterTouchEventForSecurity(event);
 				if (!isTouchAllowed) {
-					fireEvent(TiC.EVENT_TOUCH_FILTERED, dictFromEvent(event));
+					fireSyncEvent(TiC.EVENT_TOUCH_FILTERED, dictFromEvent(event));
 				}
 				return isTouchAllowed;
 			}

android/runtime/common/src/js/events.js

@@ -62,7 +62,7 @@ Object.defineProperty(EventEmitter.prototype, 'callHandler', {
 
 		// Bubble the events to the parent view if needed.
 		if (data.bubbles && !cancelBubble) {
-			handled = this._fireEventToParent(type, data) || handled;
+			handled = this._fireSyncEventToParent(type, data) || handled;
 		}
 
 		return handled;
@@ -92,7 +92,7 @@ Object.defineProperty(EventEmitter.prototype, 'emit', {
 
 		if (!this._events || !this._events[type] || !this.callHandler) {
 			if (data.bubbles && !data.cancelBubble) {
-				handled = this._fireEventToParent(type, data);
+				handled = this._fireSyncEventToParent(type, data);
 			}
 			return handled;
 		}
@@ -108,7 +108,7 @@ Object.defineProperty(EventEmitter.prototype, 'emit', {
 			}
 
 		} else if (data.bubbles && !data.cancelBubble) {
-			handled = this._fireEventToParent(type, data);
+			handled = this._fireSyncEventToParent(type, data);
 		}
 
 		return handled;

android/titanium/src/java/org/appcelerator/kroll/KrollProxy.java

@@ -858,6 +858,25 @@ public boolean fireSyncEvent(String event, Object data, long maxTimeout)
 		}
 	}
 
+	/**
+	 * Fires an event synchronously to the view who is next to receive the event.
+	 *
+	 * @param eventName event to send to the next view
+	 * @param data the data to include in the event
+	 * @return true if the event was handled
+	 */
+	@Kroll.method(name = "_fireSyncEventToParent")
+	public boolean fireSyncEventToParent(String eventName, Object data)
+	{
+		if (bubbleParent) {
+			KrollProxy parentProxy = getParentForBubbling();
+			if (parentProxy != null) {
+				return parentProxy.fireSyncEvent(eventName, data);
+			}
+		}
+		return false;
+	}
+
 	@SuppressWarnings({ "rawtypes", "unchecked" })
 	public boolean doFireEvent(String event, Object data)
 	{

android/titanium/src/java/org/appcelerator/titanium/proxy/TiViewProxy.java

@@ -961,13 +961,9 @@ public boolean fireEvent(String eventName, Object data, boolean bubbles)
 		if (data == null) {
 			data = new KrollDict();
 		}
-
-		// Set the "bubbles" property to indicate if the event needs to be bubbled.
 		if (data instanceof HashMap) {
 			((HashMap) data).put(TiC.PROPERTY_BUBBLES, bubbles);
 		}
-
-		// Dispatch the event to JavaScript which takes care of the bubbling.
 		return super.fireEvent(eventName, data);
 	}
 
@@ -977,12 +973,27 @@ public boolean fireEvent(String eventName, Object data, boolean bubbles)
 	@Override
 	public boolean fireEvent(String eventName, Object data)
 	{
-		// To remain compatible this override of fireEvent will always
-		// bubble the event to the parent view. It should eventually be deprecated
-		// in favor of using the fireEvent(String, Object, boolean) method.
 		return fireEvent(eventName, data, true);
 	}
 
+	@SuppressWarnings({ "rawtypes", "unchecked" })
+	public boolean fireSyncEvent(String eventName, Object data, boolean bubbles)
+	{
+		if (data == null) {
+			data = new KrollDict();
+		}
+		if (data instanceof HashMap) {
+			((HashMap) data).put(TiC.PROPERTY_BUBBLES, bubbles);
+		}
+		return super.fireSyncEvent(eventName, data);
+	}
+
+	@Override
+	public boolean fireSyncEvent(String eventName, Object data)
+	{
+		return fireSyncEvent(eventName, data, true);
+	}
+
 	/**
 	 * @return The parent view proxy of this view proxy.
 	 * @module.api

android/titanium/src/java/org/appcelerator/titanium/view/TiUIView.java

@@ -1679,7 +1679,7 @@ public boolean onScale(ScaleGestureDetector sgd)
 						data.put(TiC.EVENT_PROPERTY_VELOCITY, (sgd.getScaleFactor() - 1.0f) / timeDelta * 1000);
 						data.put(TiC.EVENT_PROPERTY_SOURCE, proxy);
 
-						return fireEvent(TiC.EVENT_PINCH, data);
+						return fireSyncEvent(TiC.EVENT_PINCH, data);
 					}
 					return false;
 				}
@@ -1808,7 +1808,7 @@ public boolean onTouch(View view, MotionEvent event)
 				String motionEvent = motionEvents.get(event.getAction());
 				if (motionEvent != null) {
 					if (proxy != null && proxy.hierarchyHasListener(motionEvent)) {
-						fireEvent(motionEvent, dictFromEvent(event));
+						fireSyncEvent(motionEvent, dictFromEvent(event));
 					}
 				}
 
@@ -2060,6 +2060,24 @@ public boolean fireEvent(String eventName, KrollDict data, boolean bubbles)
 		return proxy.fireEvent(eventName, data, bubbles);
 	}
 
+	public boolean fireSyncEvent(String eventName, KrollDict data)
+	{
+		return fireSyncEvent(eventName, data, true);
+	}
+
+	public boolean fireSyncEvent(String eventName, KrollDict data, boolean bubbles)
+	{
+		if (proxy == null) {
+			return false;
+		}
+		if (data == null && additionalEventData != null) {
+			data = new KrollDict(additionalEventData);
+		} else if (additionalEventData != null) {
+			data.putAll(additionalEventData);
+		}
+		return proxy.fireSyncEvent(eventName, data, bubbles);
+	}
+
 	protected void setOnLongClickListener(View view)
 	{
 		view.setOnLongClickListener(new OnLongClickListener() {