Fix memory management issues

Change-Id: Ic276eb376e9d5246c3f366c6c57449e47cfce840

debug log improvement

Change-Id: Ic8c806cdb590991ba8bc0e523f625648c991e960

Customization of value reading

Change-Id: Ic89b14e514c4aea088da1a2e22698d0a2489087f

Add to check DA profile in the flick gesture with three fingers

- On Tizen 6.0, three fingers flick gestures do not work at all.

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

Add reading for reaching the last item in dropdown list

Change-Id: I9d4831e9ce7f1ee9eb30efb61f8ebe87111d5c87

Stop sending HighlightedObjectInfo in null context mode

When control is handed over to a non-atspi application (e.g. DALi on Tizen
6.0), the highlight stays on the old application (non-atspi apps are opaque
to Screen Reader and they use their own highlight). This may cause the old
app to keep sending org.tizen.GestureNavigation.HighlightedObjectInfo D-Bus
messages (in parallel to the non-atspi app, causing a race condition), which
in turn may cause the double-tap-and-hold gesture information to be sent
with wrong coordinates.

Change-Id: I5129dd5bc49edcaab29e8e76ccf1e5cd177fb306

Disable default label when old context root object became new context current object

Change-Id: I8852a8c53181039b8a2c4a3be1a2373cfcdd0b14

flat_navi: Support returning directly to the old root

This functionality can be used with drop-down lists, where after dismissing
the list, navigation should resume at the drop-down button.

Change-Id: I067c6a5501e14ea1d46c3d1ab1f30fb1c355450f

flat_navi: Handle focus chain end in drop-down menu

This patch tries to detect the focus chain end scenario in a drop-down menu and
close the menu, before falling back to the default looping behaviour.

Change-Id: I0f9fb34825954349b6644a83e1c58e74ed3a3905

Do not cache window extents

Change-Id: I4aefde951e99f930de8d933e24d3913d492d2cb3

Fix for toast popup reading

This patch allows a transient notification UI element (like toast popup)
to be read without inducing navigation context change. It requires,
the UI element to be given an ATSPI_ROLE_NOTIFICATION by application developer.
This is analogous case to the requirement of seting ATSPI_ROLE_TOGGLE_BUTTON role
for button using custom tizen theme style.

Change-Id: I373e6f700d28364967d256af280e33de34083b36

app_tracker: Fix to null context handling

App tracker root should set to null when switching to null context

Change-Id: I61f617c87b934b203148ac01197f801d2bdcb870

Merge "In new UX reading of current value is part of state phrase" into tizen_6.0

granularity_read: type correction and optimization in _valid_character_check()

Corrected variable types and in _valid_character_check() function
and optimized number of strlen() calls in the function

Change-Id: I98dd720ea5d650bd5ffdec348cacfc04b1653423

Do not try to consume gestures in null context mode

Screen Reader tries to perform the gesture on the special null context object
(currently, the desktop). The DoGesture call in such a case will always fail
(and count as not consumed), so there is no need to even attempt it.

Change-Id: I59f62b7dcb2235852a6960099091976ae79865e8

In new UX reading of current value is part of state phrase

Change-Id: I3850bf57b6659a8886f43f6fc71cac52331aa37d
(cherry picked from commit 9b0122cd37df8bdbc4c9d9becc7e1f5c109854c7)

Support null context

Null context allows the Screen Reader to yield control to applications
whose existence it is not aware of, like DALi/NUI apps. The null context
is not actually a NULL pointer, but a special desktop object, because
the code expects a non-NULL pointer in so many places.

Change-Id: I41901899a84315cc92507c09e8471195b721ede3

Limiting locking of display to completion time of last utterance

Change-Id: Icfabaad91ee4e365e3b31bae72fc3d42c7108c5c

Do not read "Image" when highlighting a Gengrid item

Change-Id: I1f23c7b5a65fb0326ba541b4225905728d6301f3

Removal of description trait for hoversel, inwin and combobox

Change-Id: Ide2287050df7d4ac6e157bc9621c74c8f71487b9
(cherry picked from commit 540191fead2a710ba7436d52c95811c2aa951ea9)

Changes to internationalization according to new UX guidelines

Change-Id: Ibb3578f5a8a7271af2d1880444344586f4675829

Allows recursive composition of state and role traits through the controlled objects

Change-Id: I2f3577dd906a927e88aa5623f8bbb3cdf98389f9

Introduce consistent method for composition of extended description

Change-Id: I8bc5411a90ef3cf2ed6051d806c83738ee9c7ec0

Joining slider percentage value with state trait

Change-Id: I108996af10c7bbc7111ec5cb5641deaae38c1e07

Remove reading "in-progress" when progressbar reach maximum value

Change-Id: Idf3cb577e2425342e89fe91b47fbe16783a09efc

Read scroll bar information from value interface

Old implementation reads information by analyzing
widget structures. This causes problem with scroll
bar widget in nui library as this one may have
multiple "visual" elements that are inside containers.
As a result nui scroll bar widget always have two
children.

This problem was solved by using value interface to
inform screen-reader about number of visual elements
and position of selected one.

Change-Id: Ie2292bd6d80a274db5ee8751801e9b566de9e623

ATSPI_ROLE_CHECK_BOX state reading handled independently of ATSPI_ROLE_RADIO_BUTTON

Change-Id: I5b82fea47c0eea7831ecb9693c91c42e8c7c652d

Remove unreachable code

Change-Id: I0e8edd0815a62d44afe0d39b1b8090cca306beeb

Removed reading object name when checked state change

Change-Id: I05f84146fec648055fea00c672d04fc0ab8198fa
(cherry picked from commit 9ce17837b9d988df07bd1c49ea564769c6cbab3c)

Unification of translation for key IDS_ACCS_BODY_DISABLED_TTS among variants of English language

Change-Id: Idcc7c3201296d2a891c3a9c3e102c3b0cd0dd9fc
(cherry picked from commit fe5bad2b9c6d397b6d179b4f5780d9602e68c711)

Read COMBO_BOX role only as "Dropdown list"

Change-Id: I73ab66303c3b2bf545ef1399a98206416f27e796

Read "Editing" as state and not description

This is a UX change. "Edit field, (pause), editing"
becomes "Editing, edit field".

Change-Id: Ia6e9c2ff84a129d9b0be6eb933965598c28a3c88

Modified reading description for slider according to new UX

Old UX: "Slider, $value%, Flick up and down to adjust the position"
New UX: "$value%, $name, slider".

Change-Id: I4f3ab5db39baab2b93fd5cdcad2ef71ac42cd044

Removed omission of role reading for switch-type buttons

Also cleaned up redundant Slider role handling.

Change-Id: I8fed9de414b6960f890a66fa1ef28f87fffd084f

Moved state information reading before the atspi object name

Change-Id: I757fec8c4e3025a2249f8b90d15fd55732427eb9

Handle HEADING role similarly to LIST_ITEM

The new UX mandates Genlist group items to be read as "Heading".

Change-Id: I1290a2164fc19783737ef34243ec1ead1399cc28

Give chance to focus widget after accidentaly entered move slider mode

Change-Id: Icc24c4747212f993bcaca5a0af747c115b037eef

app_tracker: use root if there is not default label object

The at-spi2-atk does not provide 'GetDefaultLabelInfo'.
The Flutter toolkit is working on the at-spi2-atk.

If Flutter window is activated, screen-reader is not working
with the Flutter window for the '1 finger swipe'.
The screen-reader is working with deactivated window.

It is because that 'on_timeout_rebuild_navigation_context'
does not call 'view_content_changed' which calls
'flat_navi_context_setup' changing context's root and current
So the context root is not changed.

The patch is calling 'view_content_changed' using
the Flutter windowk before the at-spi2-atk provides
'GetDefaultLabelInfo' interface.

Change-Id: Icf3a72a9203345857f1682706bfbc6320d0ca7b6

app_tracker: set timer variable to NULL before return FALSE

First of all, it is the correct way to use of timer.

If a toolkit does not support the default label,
then it is not able to make an object have focus (highlight)
because app_tracker_context_valid_check in _focus_widget returns
APP_TRACKER_CONTEXT_NOT_VALID.

Change-Id: I6b2db7b8afd7598d63820121725ac11a22281be0

screen_reader_tts: added missing error check on tts_play call

Change-Id: I4798ffec806b69f4a94f7c868edbcce38523f388
(cherry picked from commit 41ce96e1d8f370bd5a1a0a4690756e95e5dc1193)

window_tracker: Memory leak fix

Fixed memory leak in _window_append function return with failure.

Change-Id: I4d531ec9fa5bdb292f431c3c45399d591eee5bb7
(cherry picked from commit 8b3dddab9b5d8e58912c2780550d4d978f19b654)

Fix memory leak

* Memory allocated by atspi event must be manually freed.
* g_object_unref objects returned by _directional_depth_first_search.
* Remove memory leak in navigator.c
* Remove component leaks.
* Remove memory leak in _focus_widget
* Fix memory leak and memory corruption in app_tracker.c.
* Remove memory leak from screen reader spi module.
* AtspiDevice events must be free with g_boxed_free.
* Free atspi relation set.
* Free memory allocated by atspi_accessible_get_component_iface.
* Remove leak in flat_navic.
* Remove memory leaks from smart notification module.
* Remove memory leaks from spi module.

Change-Id: I76e3c18f2b6bb1d1d6008c68a8b32587779a8f50
(cherry picked from commit 188d7c4ba7f56a50b63eca4cff8fe3974358ee54)

Add privilege 'network.get'

wifi_manager_initialize returns an error becasue screen-reader does not have
a privilege. it is not able to launch screen-reader on more stricted product.

Change-Id: I6fdf4baec7409f8a4b2cdae9f2c9593daddb42d3

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

Change-Id: I831a22dc01f04278c453398926af0c9e65b9aebd

reading_composer: Added popup Alert sub-role handling

Change-Id: Ie76ef2029977a623d5fcb42154e16fc021d2c4cf

navigator: restore slider behaviour

Old functions _value_inc and _value_dec called atspi set_value
unconditionally, so restore this way. Also while at this,
delete "return" statement from end of void functions.

Change-Id: Ib0a51cad8b43e3792a2746bbceb4f3117a785034

navigator: refactor slider code

Refactor functions _value_inc an _value_dec, which
are called after OneFingerFlickUp or Down on slider.
Also while at this, add check for minimum increase
value, if it is zero then calculate increase value
to be 5 percent.

This commit needs fix 1d2dbed4c8cbd1ef80e636ed96d45f0a9fe2fa2d
"elm_slider: add A11Y value interface"

Change-Id: I26f4d2d229ac971a6966348c503f8427bdc3e1e1

Do not notify reading status to defunct objects

Change-Id: Icf5ac4adfb2487454d379ae21a14c8145101163e

reading_composer: fix slider reading description

When slider values are not initialized or are set to zero,
this can result in division by 0.0 that can give unpredictable
values. Add some checks before division.

Change-Id: Ib269c962519191fcc3699076542ca021228cd1ef

Provided third-party licensing information in an organized way.

Change-Id: Ice57dfb4c877a7a2da0ccfdf181c46734734a1a4

Add option to generate compile-commands.json

Run local build with --define "compile_commands 1" option to
generate compile-commands.json required to build mockups.

Change-Id: I749edefe0550665ccc660546b41f241f2020e3bc

Mockup generation

Change-Id: I8cbca1bf1c3122b7b9491d2cebfea5b3f12e2c10

[screen-reader] Remove subroot window when respective window is removed

Sometimes, state-changed:visibal signal does not come and subroot can not be
removed. Use window remove function which is called from app tracker on
state-changed:showing signal too.

Change-Id: Ie9cfd418ff96e9d0da1fd0f30b4cf3007eacb65d

[screen-reader] Avoid double free of hash's value

Freeing reading_info here leads to double free, as g_hash_table_unref also
tries to free this memory.

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

Add reading composer tests

Unittests for internal functions:
 * generate_description_trait
 * generate_role_trait
 * generate_state_trait

Change-Id: Ia2054792ffa99f50fe639a9e8d48bd4a07fa1324

Support unittest build in spec file

Run local build with --define "unittest 1" option to
build unittest package.

CMakeLists.txt were partially rewritten to support
installing and packaging libraries and binaries.

Change-Id: Ia54ad687f3b478346ab808813de644ee6e9da1d4

Changes that allows built unittest files

Clang was not able to build project as LC_ALL was not defined.
Remove _GNU_SOURCE from headers.
Internal structures must be moved to headers to compile mockups.

Change-Id: Ife532459f128a46e804c655741a10f1bf111aa29

Import final changes made by R.Cybulski in cmock

This commit fixes also error:
  "unable to load <mockup_function_name> function symbol"

Change-Id: I074f634cc80e5dfeb0e13af6483506e98ca8e825

Imports final version of tests made by R.Cybulski

This commits imports final version of generate_wrapper.py.
Replaces mockups from original commit with new versions.

Change-Id: Ied824e80e25d544528f0c058c62c4918d70c5f3c

Remove warrning during compilation gtest

Change-Id: I105e5e0ecc492d414cd1781e8d3325ed405681fe

add googletest / c-mock sources

Change-Id: I44ab01fc770fcb98137d8842b5f0b3c5efd3d5c0

Remove useless code from reading composer

Change-Id: Idf493f192ca61fbd162dc9c198f3de4c4914ff8b

reading_composer: remove unreachable code

This patch is removing unreachable code. That's it.

Change-Id: I0422d2069fbc186170adb0d34f5287c2d3203607

Build script refactoring

* Add PIE options to all executables produced by cmake.
* Change SCREEN_READER_FLAGS to be compliant with
  TARGET_COMPILE_OPTIONS format.
* Avoid passing screen reader options to test framework.
  By default screen reader specific options were passed to all
  targets built by cmake files. It is not desirable behaviour
  as we also will build test framework.

Change-Id: I06dc05ba336a5ad6f016ee675b153f4b7b6a34ae

Fixed potential dereference to NULL

Change-Id: Ia0d898727544d3dc2fe4334f0382a07899967c4c

[screen reader] Code refactoring: navigator

* change name of NavigatorData to be compliant
  with convention
* hide global connection value inside context

Change-Id: I744e72dc7bff1b1b91590ac482dddb6bc897bb5e

[screen reader] Code refactoring: tts

* change function prefix from tts to tw (Tts Wrapper)
  as tts is reserved by system service
* reduce number of global values in tw module

Change-Id: I38a30734f79fbf53ddb316071464eba126a93336

[screen reader] Code refactoring: vconf

Initlization log was shown each time vconf module was used.
Move initialiazation log to proper place.

Change-Id: Idd6a3cd8cca11dc01c46478a0c07908221401e03

[screen reader] Code refactoring: reading composer

* simplify implementation

Change-Id: I7ad86ceb01148c981b89b133819ca71cda07c999

[screen reader] Code refactoring: screen reader vconf

 * remove "extern" values related with vconf module
 * change function prefix from vconf to vc (as vconf
   is reserved by system library)
 * remove dependency from screen_reader.h

Change-Id: I6a9b38ddb053a14444e3d14f02e4b97ad78b83cf

[screen reader] Code refactoring: split navigator module

 * functions related to "text to read" generation were
   moved to reading composer module

Change-Id: Iba869ddaecf6a0dfa70c89de69df4652083a09b4

[screen reader] Code refactoring: screen reader spi

* reduce number of global values inside screen reader spi

Change-Id: I90605a5b7c45baf0ba86addb04fa929614392ce5

screen_reader_system: fix potential overflow

Fix potential integer overflow.

Change-Id: I5088b07f96b05562c7bfd042cdec421c8625cf09

app_tracker: remove dead code

In function _on_atspi_event_cb parameters are checked at
begin, so there is no need for re-check them later.

Change-Id: I5db5acf11cb05f83a305a776675382dae658169d

screen_reader_system: fix dereference error

In shutdown function, variable system_data was freed and
later dereferenced.

Change-Id: I6727bb9134971c107b7a0e22c81020867b375795

Merge "navigator: remove global var sym_hash" into tizen

navigator: remove global var sym_hash

Move global var sym_hash inside function symbol_lookup
and replace uses of sym_hash by call to symbol_lookup.
As it is used also by app_tracker.c, move this into
separate file symbols.c

Change-Id: I0b55243262ca2fdc7d58c5a0bea8fd17e31526e5

Merge "Resolve build errors in 64bit architecture" into tizen

[screen reader] Code refactoring: window tracker

* reduce number of global values inside window tracker

Change-Id: I15a482ad0d3d735e75e2d486d9b1e486f397934d

Resolve build errors in 64bit architecture

The mismatch between printf format and type resulted build error in 64bit architecture.
The 'sizeof' function returns 'long unsinged int' in 64bit architecture.

Change-Id: I0a49cbe8e4e84babc88e6b84ee77aa4a5e3d418e

Removal of X11 related code

* X11 support on Tizen platform expires with upcoming release

Change-Id: Ia3bee9ed0a162457e666711ea988af835ba1a85f

[screen reader] Code refactoring: keyboard tracker

* reduce number of global values inside keyboard tracker

Change-Id: If77a4cebfd74dbea715afc19c53629deb2b6db16

[screen reader] Code refactoring: app tracker

* reduce nuber of global values inside app tracker

Change-Id: Id6ca05453b40275801f4030e1ecf7f0912df5512

[screen reader] Code refactoring: screen reader system

* reduce number of global values inside screen reader system

Change-Id: I4ae595311daaf6cfad21559fe267d3d8542f27fd

[screen reader] Code refactoring: dbus direct reading adapter

* reduce number of global values inside dbus direct reading adapter

Change-Id: I4d62955264a24a2fd457dbd837e4dd2b3b61bd8f

[screen reader] Code refactoring: navigator

* reduce number of global values inside navigator module

Change-Id: Ibed397417a442d6745c2477eaaff9b0b60dda943

smart_notification: Update resource path name

Update resource path name so that sound files can be found.
This should unify sound file search across different
profiles.

It's cherry-pick of fa31aac234aa2c3a667a45bd2f2122d03dac6427
from wearable repository, with commit message edited.

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

Navigator: Add Space in symbol table

It's cherry-pick of 9bef591f339535ead3e267e76849f5a59a797127
from wearable repository.

Change-Id: I90300ca9d2376ff1f58a16353cec178a05a370a3

[Screen-Reader]: Add Symbol reading

Summary:
Symbols can have different reading hence tts engine does not read
certain symbols and read few of symbols in a different manner.
In this patch we address this issue by providing reading
to most symbols.

It's cherry-pick of 7f6d90efefcf5aebf23c367da6c5e87a64393a2a
from wearable repository.

Change-Id: Ie16ec43cd1dbc9a4ad695484edd40ca4c1a6a96b

granularity_read: fix memory leak

Fix memory leak. Also while at this, simplify code and
prevent infinite loop for non-valid chars at text end.

Change-Id: I47d24f3c2a516d44b6df457faca449c62b633b44
(cherry picked from commit 7c4706183e3d0a1ca58a6eac1da47592fc594be5)

fix: memory leak

coverity cid: 1134382

Change-Id: Ia0263d3c4e996f4468a990d2c3700dc8178eff03

Refactoring: get_read_command_from_queue

Simplified code in get_read_command_from_queue function.

Change-Id: I91e07899a4261d8d84a7c78b7a4e7c6ca0c987fe

Fix: null pointer dereference

Change-Id: Ib66751bd9704909a0912804dc5ca88a58663e0a2

Ommit usage hint for disabled UI elements

Problem:
In elm-tizen-demo-mobile > Check for disabled widgets the following hint
is read "Double tap to select", which is missleading for the user
as disabled widget cannot be operated with gestures. Checks against
the ENABLED satate has been added before individually for some widget types,
but it seams to be a general rule not to read usage hint for disabled widgets.

This patch adds global rule to not read default description for disabled widgets.

Change-Id: Ia2f054fd84511808c8d81f3b9777a8d84f3e4f96

Merge "Gestures support by app is checked when view content is changed" into tizen

Gestures support by app is checked when view content is changed

This patch allows to update the policy of forwarding gestures
directly to application each time the navigation context is built,
not just when window:activate signal arrives. The policy is configured
basing on the root object of navigation context.

Change-Id: I1861021962e68a15db9882ef1d0ac6b17aeee22d

Fix: memory leak

Function purge_commands_from_queue did not free memory
allocated under qc ptr.

Change-Id: I011573ae6de17d0ed79b8a8c941662c6095d34fd

Fix: null pointer dereference

There is no mechanics to force client to set on_utterance_end
callback in screen_reader_tts.c file. This gives opportunity
to dereference null ptr.

Change-Id: I6e8d24eb2dd6bf9a45e36b1d5476c39668ac1903

Fix: invalid pointer usege

If tts_create fails the code will try to dereference invalid ptr.

Change-Id: I7e4637cfc1a5a3b2e553bdec68b0da4ae96fcb81

Added check for empty text when adding reading command to queue

Change-Id: Ibf753d614f42b440643f5bfd4748a57b4b182199