Add missing nullchecks

Change-Id: Iaf0c51165bfb4aff6484cd631b2fa761f9e969d8

Provides simpler and less vulnerable filtration of unwanted mouse events while dragging

Fixes: https://review.tizen.org/gerrit/#/c/platform/core/uifw/e-mod-tizen-screen-reader/+/282157/

Change-Id: I03ecc5632bc4781798c3ede14b0bbf180a60f430

Ignoring mouse events not participating in drag gesture

Change-Id: If864f916b215f36b973e0d6eab6fd83850627d00

Fix to set the correct gesture position in multi-touch

- When user tries to zoom out/in an object, sometimes the object gets
bigger than the desired zoom behavior.

Change-Id: Iafef71c96c03f7c1aba69463eec633ae8a732865
Signed-off-by: Seoyeon Kim <seoyeon2.kim@samsung.com>

Quick fix for quick 2-finger hover gesture

This patch adds an extra emission of the 'ongoing' gesture signal before
'end' in case there wasn't one.

Change-Id: I877407bc4a120694411ce7d86f5445d19761066a
(cherry picked from commit 9792b9f611bc971fc856c1bac7e9a29430ab231f)

Send 'GestureDetected' with a resource id

The resource id is unique value in the entire system.
It will be used to get a window under the x,y position.
It will be very correct in the multi-window case instead of
tracking window extents and its stack.

Change-Id: I03f75470b4eea7316f76eab21ba87b6b449c5662

Removal of dead code related to invalid_area_size

invalid_area_size.bottom and invalid_area_size.top are unsigned int so
the following conditions:

if (aconfig->conf->gesture.invalid_area_size.top < 0)
if (aconfig->conf->gesture.invalid_area_size.bottom < 0)

are allways false.

Change-Id: I499c92e62fb3ef9f747c405b4408c2d6ecda0fa6

Introduce invalid area

This patch absorbs the forked git commit below.
https://github.sec.samsung.net/Tizen-DA/e-mod-tizen-screen-reader/commit/aae916a292f2c36f91812a5d3da87fbd04f49904

Above commit and forked git could be removed by this patch introducing
the invalid area.

To make invalid area have size, we need following change.

e-tizen-data/default/config/tizen-mobile/module.screen_reader.accessibility.src

 group "Screen_Reader_Accessibility_Config" struct {
       value "gesture.quickpanel_area_size" uint: 40;
       value "gesture.longpress_timeout" float: 0.4;
+      value "gesture.invalid_area_size.top" uint: 70;
+      value "gesture.invalid_area_size.bottom" uint: 50;
 }

Change-Id: I2995ec5356993c430ec6ff072dcba8dcc7bab1aa

Handle Quickpanel

There are different type of quickpanel as below

 - E_QUICKPANEL_TYPE_SYSTEM_DEFAULT
 - E_QUICKPANEL_TYPE_CONTEXT_MENU
 - E_QUICKPANEL_TYPE_APPS_MENU

Thf FHUB AllApps is quickpanel type applcation and
it is DALi application which does not support AT-SPI.

So if the AllApps is on top, then screen-reader should NOT
consume 'GestureDetected' signal.

Refer to follwoing for more cases;
 - https://github.sec.samsung.net/TizenNativeUI/Graphics/issues/1281

So screen-reader module needs to notify quickpanel state.

Change-Id: I832624873370ba47c693c5e8de9bb34a3c63be7e

Create global context structure

Change-Id: Ied639a8421db9aa4ad565f4da27996fafaf028b1

Encapsulate '_e_mod_config' global variable

Change-Id: Ib45cc4d5345ed9d9f49ab851b3eab32ed7b493cd

Make non-global top-level variables static

Change-Id: Ief69383e2de9b2730fea748fe5823ddb0f5b368b

Encapsulate '_eina_log_dom' global variable

This is a similar approach to how 'errno' is implemented in most (thread-aware)
C standard libraries.

Change-Id: Ie1efa3a18e5999822db631689c3686cfbae6ff32

[e-mod-tizen-screen-reader] Emit hover events of ONGOING state in interval of 200ms

On hover, gesture update is sent continously which degrades
performance. Screen Reader is bombarded with continous unneccesary
updates about hover which causes performance issues for feature like
scroll audio feedback. Limiting hover events to 200ms intervals helps
to fix this.

Change-Id: I0f9e546d099b0eca8e3dfef033f8fbfadc4c3376
Signed-off-by: Prasoon Singh <prasoon.16@samsung.com>
(cherry picked from commit 25e6aa0df955a290b10363257bed3379ecfbd763)

e_mod_main: follow e20 policy using e_dbus_conn

There is a policy of enlightenment using e_dbus_conn.
And there is a request to follow this policy. That's it.

Change-Id: I5a659cbc39e713ff5b20275f931820fa9feef9dd

Handle mouse pointer movement when using screen-reader

This patch depends on a commit in Enlightenment:
  - e_pointer: change cursor moving APIs to E_API
  - Change-Id: I104fe415d96e95740c921170a2f609f89f9ef34e

Change-Id: I8622f1f36308012276ee5c3087986d77d116da6e

Do not consume mouse event if using mouse pointer

This patch fixes a bug where the mouse pointer becomes frozen when
screen reader is turned on.

Change-Id: If09e04fd881c3808ce353317e71e7ecd31fea3a5

e_mod_screen_reader_gestures: fix dereference null return value

Change-Id: Ie23500c8795e95f9d9ab11817acb9721d0665ae2

Screen-reader: Two finger move gesture sensitivity increased as its
difficult to scroll, flick with two fingers currently.
PLM fixes: P200407-01580, P200311-04050
Signed-off-by: Prasoon Singh <prasoon.16@samsung.com>
Signed-off-by: Shilpa Singh <shilpa.singh@samsung.com>
Change-Id: I6f3b8f810523476176e759ee18085bd18def5128

[e-mod-tizen-screen-reader] Reduce the timeout threshhold of flick to scroll

As mouse move is emitted only if difference between down and move
event time is greater that 100ms. Due to which when flick is done
scrolling could not start. Only if drag is done scrolling starts.

Change-Id: I4fdd294ebaca579438a288b0348af8fd6132bad1
Signed-off-by: Prasoon Singh <prasoon.16@samsung.com>
(cherry picked from commit bd6f28fddbc4daa87d2a76681518bde3be14f11a)

[e-mod-tizen-screen-reader] Make opening QP easier by emiting events when touched upon
QP area.

For easier opening of QP, needs to emit mouse move events directly. Only
checking in horizontal direction is not enough, as second finger always out of QP
area. So, checking if y cordinate of fingers if lying in QP area in vertical
direction makes more sense.

Change-Id: I935bba628bf05b2f29788a61ca4c6a643b8928da
Signed-off-by: Prasoon Singh <prasoon.16@samsung.com>
(cherry picked from commit b5276ad51c0c457f998976b47fba667f9a0cab02)

Merge "deregistering vconf value change callback on module shutdown" into tizen

deregistering vconf value change callback on module shutdown

Change-Id: Ie1e4595aa3870119b5bf56df8cffd8e00a6983fc

Merge "[e-mod-tizen-screen-reader] Abort flick gesture when mouse up comes with (0,0) coordinate" into tizen

Merge "[e-mod-tizen-screen-reader] Propogate multi device mouse down and up for two finger double tap n hold" into tizen

[e-mod-tizen-screen-reader] Propogate multi device mouse down and up for two finger double tap n hold

Change-Id: I5469a27ec8b11499226cd06f0f63040b06ea9e35
Signed-off-by: Prasoon Singh <prasoon.16@samsung.com>

[e-mod-tizen-screen-reader] Update timestamp before sending hover event.

Hover sent when flick_to_scroll is true does not have correct timestamp.

Change-Id: Ib062dd02283e139b1359e0890f3f8bb3a9630e13
Signed-off-by: Prasoon Singh <prasoon.16@samsung.com>

[e-mod-tizen-screen-reader] Abort flick gesture when mouse up comes with (0,0) coordinate

In touch bezel devices when down is generated and up happens on touch bezel, mouse up with (0,0)
cordinate is sent. Due to which length threshold is fulfilled and flick gesture comes.

Change-Id: I0f634c052a4a0f394dcf92243f23e8e75f69f46d
Signed-off-by: Prasoon Singh <prasoon.16@samsung.com>

Merge "Make hover gestures near edges easier to perform" into tizen

Make hover gestures near edges easier to perform

Change-Id: I84c539d8dabe77d7a3f4062f74b7f9cf2ea4cbd1

Check pointers after malloc operations

Change-Id: Iea7c54dcb90972a684db884d0c996c302716d96b

Change behavior of e_accessibility_conf_init

Problems with loading configuration file
("module.screen_reader.accessibility") will not cause e_mod_main to fail
anymore.
In case of configuration file problems default values will be loaded.

Change-Id: If61d00c6b419ba22863e3a741dd3e97d45f856f1

Removed the code that calls e_modapi_shutdown directly from the module.

e_modapi_xxx functions will be called by the enlightenment not by the module.

Change-Id: I5c8ae7d0cbe70cb8bb958e7561dd6cca5eb643aa

module.accessibility config name is overlap.
Need change module.accessibility config name.

   Delete overlap config: module.accessibility
   New create config: module.screen_reader.accessibility
   Modiffy config group name: Screen_Reader_Accessibility

Change-Id: I21690f482012c35b8a2ec1e005b56aafbcc77d12

Fix invalid device pointer in DispatchGestureEvent

Change-Id: I937b336be11de4c0e3058dc67059ff8107435f08

Add DispatchRotaryEvent to DBus interface

Change-Id: I863b36f96d950ebde097bf77414b89a23d6e7c1a

Customizable forwarding of gestures directly to app.

This commit provides mechanism that allows app developer to request
delivery of particular gestures directly to application and by-passing
screen-reader.
The list of gesture is sent from application to screen-reader as ATSPI
attribute.
Then screen-reader forwards this list (using DBus interface,
AppGestureSupport) to e-mod which decides how to route detected
gestures.

Change-Id: I49e87a5c78ed39ac917deeade8b2f954c5bb6b9d

[e-mod-tizen-screen-reader] Use two finger longpress timeout value for two finger hover gesture

Change-Id: I7d70445ed79170d2c2eee08a0a90d9a84d466b7d
Signed-off-by: Prasoon Singh <prasoon.16@samsung.com>

DispatchDragEvent improvement

This patch modifies signature and implementation of DBus method
"DispatchDragEvent" so gesture can be performed with multiple pointers

Change-Id: Id91f5e4c20fd9c1477f78d6c9018668aef8b1c76

[e-mod-tizen-screen-reader] Config values which are check for two finger scroll are not actually added.

Change-Id: Ib028ecebc35ae4f31d96665c3a6d45429ad83bd9
Signed-off-by: Prasoon Singh <prasoon.16@samsung.com>

[4.0][Pulse][e-mod-tizen-screen-reader] Not invoke gesture callback on zoom mode.
[Model] Common
[BinType] AP
[Customer] OPEN
[Issue#] Not invoke gesture callback on zoom mode.
[Request] UX
[Occurrence Version] N/A
[Problem] Not invoke gesture callback on zoom mode.
[Cause & Measure] Not invoke gesture callback on zoom mode.
[Team] UIFW
[Developer] chulheon jeong(ch.jeong47@samsung.com)
[Solution company] Samsung
[Change Type] fix

Change-Id: I549a655a141243d4300a118f34c6266bc1ca989d
Signed-off-by: ch.jeong47 <ch.jeong47@samsung.com>
Signed-off-by: JunsuChoi <jsuya.choi@samsung.com>

[Fix] touch event unpair issue

This patch resolve issue when ECORE_EVENT_MOUSE_BUTTON_UP
doesn't come to other registered ecore event filters

Change-Id: I7c8a763d0eafb524e218293440458bd1ed2746ec

Fix "undefined reference" error

This patches changes function name, so it matches its prototype.

Change-Id: I3f41fc31961b31bf7abd24c020bea63caa753768

Add accessibility configuration with eet

Change-Id: I8a4a2491f2bb3cb46b7756b90532c5e73fb2cb70

Add filtering of events when US menu is shown

Universal Switch needs information about mouse events
when its contextual menu is shown. Because it does not
receive any events directly e-mod sends them through DBus.

This patch also renames some functions.

Change-Id: I081043f10b3fa8c24d9a1dbb06a721013ae69f51

Add exception handling in case of memory allocation fails

Change-Id: I05bdd0d15f8c71330f79e6939e0c742f3a165d96

Dispatching key events

E-mod can dispatch key events via dbus method call.
This approach enables synchronous dispatching key ecore events.

Change-Id: I7fb11d4eb93ed1bec71da6dd38dfb427ffcafc3d

Refactor in dispatching gesture events

1. Error messages are more descriptive.
2. Refactor of transform_coordinates and correct_coordinates functions

Change-Id: I7a2e4e3d23070581bcbcaea6a223c1f318cb41ee

Add transformation of coordinates

When screen is rotated coordinates used in DispatchDragEvent
and DispatchPinchEvent are transformed to new orientation

Change-Id: I4f58567a6989c0daec9fc7a0ed322276aa29382b

Refactor of e_dispatch_gesture_event.c

Change-Id: If40691685f0552da31b168c1c940ee445500ea4e

Pinch gesture with parameters

Pinch gesture enables both "zoom in" and "zoom out" functionality.
Because of generic pinch and drag gestures other functionalities
are needless.

Change-Id: I9bc42ffed9d14f1616784f806379805f3154531d

Make e_dispatch_gesture_event work again

It is not possible the root reason why e_dispatch_gesture_event does not work.
This patch set is following what e-mod-tizen-devicemgr does.
The _e_input_devmgr_generate_touch_event generates touch event using ee_win.

Change-Id: I552bf94b590ff29ed1e660e4ce7a3de9bbdb1f01

Send reply when drag gesture is finished

Reply messages are sent when all events are placed in ecore queue
not just after parsing arguments

Change-Id: Id72543e11400e1392dffcf824465c628d8c8d3ed

Update e_screensaver before consuming touch event

There was an issue that the screen saver works during user interaction. It is
becuase that screen reader module uses ecore event filter, and consumes touch
event. So the e_screensaver did not get the touch event.

This patch set is using e_screensaver_notidle function to inform e_screensaver
about touch event before screen reader consumes touch event.

Change-Id: If47a61b765030641a526adc84030387905d3db6c

Implementation of dispatch rotation event functionality.

Change-Id: I8a306ec61faa15a7443b9f517b04742d7c380b59

Fix for dispatching drag event

Add guard during execution of drag event.
This will prevent from crashing when another drag event is being started
during previous is being executed.

Change-Id: I65b92023c9811eb17a3099758b882dcf5b8d3e6e

"Touch, hold and drag" gesture implementation

Change-Id: Ic086968b0326106ac8656a7977c79619c1a62fd1

Drag gesture implementation

New dbus method which allows to dispatch drag gesture
from one point to another in defined number of steps

Change-Id: I42b29c1f0cb1943c9c4f13bfc6908416597e5406

Add BackButtonInterceptionEnabled informing about pressing back button

Change-Id: Id18de1a60f4c1a5d4d2b251c78110a1fc9d0298b

Remove unnecessary overwriting code

Change-Id: I3203e062c7159e059ba534930fd0f80317f29ff2

Merge "Use delta value for the "Drag & drop" gesture" into tizen

Add interception of key event

Module can intercept key events and send information
about them via dbus

Change-Id: I5e5f56ac3467e6bc7b22b224339f0c3d82e85b4b

Use delta value for the "Drag & drop" gesture

If there is a highlighted object, the "Drag & drop" gesture uses the geometry
value of the highlighted object.
But the geometry value was used only for 'mouse button down' event,
and was not used for 'move' event.
So we are using delta value which is difference between highlighted object
geometry value and 1 finger touch point on the screen for the "Drag & drop"
gesture.

Change-Id: Icfa9879850c58e941799e97fcef74149f53404b7

Adding method and signals used by Universal-Switch

This patch adds ScreenSwitchProviderEnabled method to eldbus API provided by the module.
The method enables/disables ecore event filter capturing mouse events from user.
Information about captured event is sent by eldbus signals to universal-switch.

Because of added filter particular device id is set in gestures created by module itself.

Change-Id: I7fa8389a9604b64f0f9068a508d74a4f33aed249

Support API eext_gesture_event_dispatch

This patch set make eext_guesture_event_dispatch of platform/core/uifw/efl-ext
work(https://review.tizen.org/gerrit/#/c/127914/).

Change-Id: I64a804d39ba351912a90ba1e4f00faad77d1523a

Add "ObjectNeedsScrollGesture" method

The screen-reader module changes x,y information of 2 fingers hover gesture
if highlight object needs it.

Application side could use this feature setting attribute as below:
 - elm_atspi_accessible_attribute_append(btn, "gesture_required", "scroll");

This patch set depends on:
https://review.tizen.org/gerrit/#/c/142105/ (elementary)
https://review.tizen.org/gerrit/#/c/142108/ (screen-reader)

Change-Id: I0312a748af5bb514dc085f97c5e1ee7f7acad5a5

Make reliable module

E_NEW could return NULL.
So screen-reader module is checking return value of E_NEW before using it.

Change-Id: I755ec02a5a56ff98946d569bb41f3db63d58d6c6

Remove identical branch

Change-Id: I5df69444ffdb4d5cea499f4c0ee04ad078b4f3ac

Remove memory leak possibility

Change-Id: I9c9ebb825c5a93a82e840e3ba14dd67684e75dc4

Check vconf value at init time

The _elm_atspi_bridge_init is moved to elm_run from elm_init because of
security issue.

The at-spi-bus-launcher is launched by D-Bus activation.
The "org.a11y.Bus" which is used for the D-Bus activation would be used after
the elm_run is called.

The at-spi-bus-launcher launches screen-reader AT client.
The screen-reader AT client requests that screen-reader WM module works.
Before this point, the user input is not consumed by screen-reader WM module.
So it is possible to control the first appliction (home-screen) by user input.

[Vconf Dependency]
The enlightenment already has dependency on vconf as below.
So this patch set does NOT need more library loading time which could cause
the late launching time issue.

sh-3.2# cat /proc/313/maps | grep vconf
b5f2e000-b5f32000 r-xp 00000000 b3:1a 6053       /usr/lib/libvconf.so.0.3.1
b5f32000-b5f41000 ---p 00004000 b3:1a 6053       /usr/lib/libvconf.so.0.3.1
b5f41000-b5f42000 rw-p 00003000 b3:1a 6053       /usr/lib/libvconf.so.0.3.1

Change-Id: I0278a67e018d666325a93fb37e7eb66c95647e64

Support selection mode

Double tap & hold with two fingers to enter the Selection mode.
If the text input filed is changed to selection mode, then we do not use two
fingers to scroll. The two fingers gesture is used as below:

 - 2 fingers flick down: select all
 - 2 fingers flick up: copy
 - 2 fingers flick left: cut
 - 2 fingers flick right: paste

Change-Id: Iede0729340f352249e1f489005b66f2bc4247b37
Needs: https://review.tizen.org/gerrit/#/c/122452/

Emit 2 fingers double tap and hold gesture

The "2 fingers double tap and hold" gesture will work to change between
selection and edit mdoe.

Change-Id: I0ec772d429f406971ee27d5949c6a160693094d8

Not init sub module if it is inited already.

The screen-reader is enabled again without disabled,
if the screen-reader is terminated abnormally, and launched again by at-spi-bus-launcher.

Change-Id: Ic4f7ee446f741325c3e0073a17ebad0e609d1d8a

Using %license macro to install license file instead of using manual command.

Change-Id: Ifc6b663e13b0613183b0c86ce7efdb93ed28c1d0

Merge remote-tracking branch 'origin/tizen_3.0' into tizen

Remove ecore-wayland dependency

It is improper that the Enlightenment (Window Manager) module uses ecore_wayland API (client side API).
This is recommendation of window manager team.

Change-Id: Id8bb0e6989aab489192077634e2821ae3c4804db

Add link flag of ecore-wayland

Loading module is failed because of following error.
/usr/lib/enlightenment/modules/e-mod-tizen-screen-reader/linux-gnueabi-armv7l-ver-autocannoli-0.20/module.so: undefined symbol: ecore_wl_shutdown

It seems that ecore-wayland dependency is removed by https://review.tizen.org/gerrit/#/c/115433/

Change-Id: I8a19ef87e337c898f1431bc7df27523d165aaf17

Fix wrong down coordinate of 2nd finger

The 2nd finger coordinate was same with 1st finger coordinate.

Change-Id: I2b896f50f2b723b8f25a33bdbd9c1453e1eb0e99

Add null check for when get window angle

   add null check before using e_zone_current_get's result variable.
   when get window angle

Change-Id: Ief734dfb94e74c0e7e535b69ea0014e0431e470f

Fix wrong down coordinate of 2nd finger

The 2nd finger coordinate was same with 1st finger coordinate.

Change-Id: I2b896f50f2b723b8f25a33bdbd9c1453e1eb0e99

Add null check for when get window angle

   add null check before using e_zone_current_get's result variable.
   when get window angle

Change-Id: Ief734dfb94e74c0e7e535b69ea0014e0431e470f

Check window angle before sending gesture event

The top e_client could not take care the angle of device.
So checking window angle at module initialization time is not proper.
This commit will check window angle before sending gesture event.

Change-Id: I36ce00431f2101bf51b3ade6cfa91442264faaba

Check window angle before sending gesture event

The top e_client could not take care the angle of device.
So checking window angle at module initialization time is not proper.
This commit will check window angle before sending gesture event.

Change-Id: I36ce00431f2101bf51b3ade6cfa91442264faaba

[1 finger tap and hold] should use highlighted object information.

- In case of rotated device, the coordinate should be changed properly.
- If 1 finger tap is aborted, the highlighted object information comes from screen reader AT-client.
  We need almost 1 sec between the tap timeout and the hover timeout. (heuristic value)

Change-Id: I080fbc521d763ff56a29c90e635222923e94a43d

[1 finger tap and hold] should use highlighted object information.

- In case of rotated device, the coordinate should be changed properly.
- If 1 finger tap is aborted, the highlighted object information comes from screen reader AT-client.
  We need almost 1 sec between the tap timeout and the hover timeout. (heuristic value)

Change-Id: I080fbc521d763ff56a29c90e635222923e94a43d

[config] reduce 1 finger hover long press timeout value

Change-Id: I875dbaa457c0c6312f786f7271efc61e4c791b62

Make quickpanel open/close easy by sending mouse down event directly at top/bottom area without timeout

Change-Id: Iadec916bea8a2669ef18e660a3b842b467dbd891

Initialization angle - CE product could have 90 by default

Change-Id: I8d75eb0208f19b70912a6c5d3730f8a8cbb13816

[feature] check if application window supports screen reader

If an window has attribute as below, the screen reader does not support the window.
elm_atspi_accessible_attribute_append(ad->win, "screen_reader_support", "no");

Change-Id: I10ee0814fba30a419d0f256fd8647b8274d76834

Merge "[e-mod-tizen-screen-reader] Making is_slider variable false on mouse up" into tizen

[e-mod-screen-reader] Check direction of flick based on angle

Change-Id: I81f905d11ba87f4dc0e5af45954f7ece003a91df

[e-mod-tizen-screen-reader] Making is_slider variable false on mouse up

Change-Id: I47c7cc6eaa0a70b08e8bb6ce471460974fece910

Merge "Do not emit 1 finger hover finished event, if drag started" into tizen

Do not emit 1 finger hover finished event, if drag started

Change-Id: I4cbb25a57504b78ed89026e4e251d72e80bc4d22

Release mouse move only if hover longpress timeout happens.

Change-Id: Ice1ce5ee83da133d4774d059c8ed382ec4d5b217

Merge "Fix multiple definition of 'is_slider','highlighted_object_x' and 'highlighted_object_y'." into tizen

Merge "Sending hover mouse move at the time scrolling. x,y cordinates will also be updated for hover in hover mouse move." into tizen

Fix multiple definition of 'is_slider','highlighted_object_x' and 'highlighted_object_y'.

Change-Id: Ibad280d05fb519a709572c4d8235d7336ef687d7
Signed-off-by: Chan Lee <chan45.lee@samsung.com>

[e-mod-tizen-screen-reader] Update  mouse down event at time of double tap n hold based on highlighted
object x,y position.

Change-Id: Ia37cbc052937a8828fb3d34918a6c16a75a0d1b1

Sending hover mouse move at the time scrolling. x,y cordinates will also be updated for hover in hover mouse move.

Change-Id: Ifbe768c8a946f8c4cb150affe78c0fef18ae67b8