evas_events: do not to cause abort by input event feed from post event callback 50/213550/3
authorJaehyun Cho <jae_hyun.cho@samsung.com>
Fri, 6 Sep 2019 12:16:24 +0000 (21:16 +0900)
committerWooHyun Jung <wh0705.jung@samsung.com>
Mon, 9 Sep 2019 04:26:12 +0000 (04:26 +0000)
New input events feed from a post event callback may be risky because it
may cause recursive calls. (T3144)
However, it is extremely difficult to cause recursive calls in this
case.
Therefore, the CRI is replaced with the DBG not to cause abort by new
input events feed from post event callback.

e.g. "drag,start" callback of elm_interface_scroller is called in mouse
move post event. If app sets size of an widget in "drag,start" callback,
then it may cause mouse move event again in the mouse move post event
and the CRI log message is called. But since "drag,start" callback is
called only once, the mouse move event is not called recursively.

Change-Id: Id1aea1dd6916a4bd723564218acc17761556b9ad

src/lib/evas/canvas/evas_events.c

index fbe91d0..49001cb 100644 (file)
@@ -36,7 +36,19 @@ static inline void
 _evas_event_feed_check(Evas_Public_Data *e)
 {
    if (EINA_LIKELY(!e->running_post_events)) return;
-   CRI("Feeding new input events from a post-event callback is risky!");
+   /* TIZEN_ONLY(20190906): do not to cause abort by input event feed from post
+    *                       event callback.
+    *                       e.g. "drag,start" callback of elm_interface_scroller
+    *                       is called in mouse move post event. If app sets size
+    *                       of an object in "drag,start" callback, then it may
+    *                       cause mouse move event again in the mouse move post
+    *                       event and the CRI log message is called. But since
+    *                       "drag,start" callback is called only once, the mouse
+    *                       move event is not called recursively.
+    */
+   //CRI("Feeding new input events from a post-event callback is risky!");
+   DBG("Feeding new input events from a post-event callback is risky!");
+   /* END(20190906) */
 }
 
 static inline Eina_Bool