Merge "Fix keyboard removal logic in _window_remove" into tizen

Merge "Set a window as active when added by state-change:visible" into tizen

Merge "Implement circular inc/dec for SPIN_BUTTON role" into tizen

Fix keyboard removal logic in _window_remove

Change-Id: I8a6322e62b3c9256b548a11bf925218582daacd1
(cherry picked from commit eddb05c76808da23d24cf5aebfdfabff84ff58de)

Set a window as active when added by state-change:visible

Change-Id: If415de57e7ea2afecea155fd6ad47815f2e83ab1

Prevent context switing when unfocused window is added

There are cases "showing" events comes after a window is focused, which causes loss of focus of the currently focused window. To prevent this, updated to switch context on focused("activate") instead of on window shown.
Also, added a condition for "NameOwnerChanged" to prevent removing the window when a widget proxy is removed.

Change-Id: I2d1b24539b8b1504e878e84b2cfc06a2ee14a4bb

Implement circular inc/dec for SPIN_BUTTON role

Change-Id: I78ad8cf129cee00ef5f605d76916399300adbcbd

Merge "Update translation for IDS_ST_MBODY_SCREEN_READER_HTTS key" into tizen

Merge "Simplify window purging procedure" into tizen

Merge "Focus to default_label after dispose keyboard" into tizen

Merge "Prevent calling _focus_widget when keyboard is on top" into tizen

Merge "Extend changing translation to all version of english language" into tizen

Merge "Add ComboBox to potentially modal objects" into tizen

Merge "Reduce the delay before start switching context" into tizen

Reduce the delay before start switching context

Change-Id: Ifdd6629d174bf87528dd45237d40e3d9d1429317

Add ComboBox to potentially modal objects

An object needs to have both ATSPI_ROLE_COMBO_BOX and ATSPI_STATE_MODAL.

Change-Id: Iefd43d570f6bc04a7ae50bd3fb3a246ae0abdddc

Extend changing translation to all version of english language

Change-Id: Ia2eac52d988b7ce61446206172f37be9ae8bc86e

Prevent calling _focus_widget when keyboard is on top

We don't want to call _focus_widget on keyboard as it won't initiate context switch thus it finds an object from the below context.

Change-Id: I94e0e0171694328860755ebbe8e65a1fb70e69e3

Focus to default_label after dispose keyboard

Change-Id: I7eb5b645346a2950f273d6ac6499c706963d0c78

Simplify window purging procedure

The checks during window destruction are mostly redundant. Also add a recursion
guard to purge_windows_list.

Change-Id: I90170a145737f742f3c77627da547903f7d8b4c5

Set description speak delay to 0

UX request that the delay between utterance of the main text and subtext was too long, the delay has been changed to 0.

Change-Id: Iaba593a603958aea4df1ec1df9de171434ced915

Update translation for IDS_ST_MBODY_SCREEN_READER_HTTS key

- Currently, only English and Korean translations are updated for
IDS_ST_MBODY_SCREEN_READER_HTTS key.
- After getting all translations, need to change the
internationalization according to new UX guidelines.
- This patch is only for Tizen 7.0 now, but we could consider adding a
separate DA profile in the future.

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

[SVACE] Add to check NULL value of `tw_get_instance()`

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

Revert "Make focus navigatable between windows."

This reverts commit a04b90495e53aeef377a9507687b448b19f4b319.

Change-Id: I48b0cb8c8780ebb78db6884cdf595ac9b2dd59a4

Remove unused features

Change-Id: I978852d2297a6fdf85e13a1bc722e431ba887d4f

Add failure checks on init procedure

Change-Id: I27b5673c6bbff789df9eb81ab2aacf41fa11fc1d

Add null check on context switch

Change-Id: I7daa865948ce6e468d21898c1f45f2a9babd4dfa

Merge "Update Symbol Lookup table for Circle, Square, and etc." into tizen

Merge "Update translations to some languages" into tizen

Merge "Add some symbol translations" into tizen

Merge "Update translations for newly added keys" into tizen

Update Symbol Lookup table for Circle, Square, and etc.

- Added some symbols to read the newly added symbols.
 : ○, ●, □, ■, ♤, ◇, ♧

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

tts: Increase log priority from Debug to Info

The Debug logs are filtered out in some configurations, in which case TTS bugs
are incorrectly classified as Screen Reader bugs. Thus, it would be advisable
to increase the log priority of the screen_reader_tts module from Debug to
Info, in order to always be able to verify what Screen Reader sends to the TTS
engine (in case there is no audible TTS output). The log priorities in other
Screen Reader modules can be kept unmodified for now.

Change-Id: I65f0e7bcfa73e8c0052df2d30e60abfb6c23294d

Update translations to some languages

- Update some translations for the following keys :
  IDS_ST_MBODY_SCREEN_READER_HTTS / IDS_MSG_BODY_ON_T_TTS / IDS_STORYALBUM_BODY_OFF_T_TTS
- Start to support three languages : Indonesian, Thai, and Vietnamese
- Modified some Korean translations for request

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

Add some symbol translations

- Currently, 24 languages are added for some symbols like ○ .
 Other languages are written in English temporarily.

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

Update translations for newly added keys

- In the past, some keys were added for release
 even though they were not translated all except for Korean and English.
- Now, some languages are updated for the keys :
 IDS_ACCS_BODY_COLUMN_P1SD
 IDS_ACCS_BODY_DOUBLE_TAP_TO_SELECT_TTS
 IDS_ACCS_BODY_IN_GRID_CAPTION_P1SS
 IDS_ACCS_BODY_P1SD_COLUMNS
 IDS_ACCS_BODY_P1SD_ROWS
 IDS_ACCS_BODY_ROW_P1SD
 IDS_ACCS_OPT_SETTINGS_VZW
 IDS_ACCS_TBOPT_CHECKED_TTS
 IDS_ACCS_TBOPT_HEADING_TTS
 IDS_ACCS_TBOPT_NOT_CHECKED_TTS
 IDS_ACCS_TBOPT_SLIDER_TTS
 IDS_ACCS_TBOPT_SWIPE_UP_OR_DOWN_TO_CHANGE_T_TTS
 IDS_ACCS_TBOPT_SWITCH_TTS

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

flat_navi: Improve reference counting

Change-Id: Ib906fb81df409f8deb0facf5c95d209e1b6acced

Skip reading role ATSPI_ROLE_SECTION only for chromium originated accessible objects

Change-Id: I6ccc5bbb9e869e6a7014d571803363c275a3b8b6

Merge "Add support for increase,decrease action when value text is set" into tizen

Add support for increase,decrease action when value text is set

Change-Id: I57d7dd80654e559f47b8951802b7df211b3cd247

Remove unused _accept_object

Change-Id: If298a989d9a4dbeeec107f348d4a90799b45a6e3

Add symbol support in direct reading

Change-Id: I309f657d2a3a527d3eb95b136ce3000074479e4e
(cherry picked from commit a81d7e839cce52f0913b832d97f48282abdaac1d)

Supports the migration script for OS upgrade

Supports the migration script for OS upgrade in Tizen TV

Change-Id: I309f3ca611c3d193890d8bf3986a588cec38b733

Revert "Add window locale role"

This reverts commit e90efc83b741c307244c8d13e51d6d2f0fc0151b.

Change-Id: I367098d6d50464973366b1385b1c06d1734fd52f

Add window locale role

Change-Id: I2c7e0c2dd48bdbcfbc90b63060385f329222d4d2

Merge "app_tracker: fixed memory management in _on_atspi_event_cb" into tizen

app_tracker: fixed memory management in _on_atspi_event_cb

Change-Id: Ia908eb4cdcc1ee5d60e641ac6925ed539566eb4e
(cherry picked from commit 112bd50320e0f1b010f635d8efcedc018766f6c4)

Use RadioButton role translation

Change-Id: I4b2bdba18b5d5beab02d4afce7c7d5577be515d8
(cherry picked from commit f1a0e65e40922a2cd11e20629f2aeb87543df93d)

Update es_US po file.

Change-Id: I1773cdf174a4fd0f9b3a8197d8dfc8f4ecca5abf

Unify switching navigation context on TAP and on ATSPI event

Change-Id: I40b5060823b5622356dc2e4236644826103183bc

Make focus navigatable between windows.

Change-Id: Iafe27b170f4bb195c6e24b93ed8f3fadc9b4ab0e

Change volume type of feedback.

TTS using 'system' volume type, so changing sound feedback's volume type
to 'system'.

Change-Id: Ifb58aa300c27f0491b785f1933f8d2d8a44dae25
(cherry picked from commit 65410b938002578b694ce32bfa35f5d8ba347fa0)

In multi-window model window should be removed from cache only when destroyed

Change-Id: I4ec493c8f436c3db547e4d4b2a52f45471727176

Null-check app_get_resource_path()

The return value is passed as the first argument to g_strconcat(), which
expects a NULL-terminated argument list, but its documentation
explicitly states that the first argument must be non-NULL.

Change-Id: I2bfab7540fd0e717007399ed5e41fece644979e0

Fix string length type as size_t

Change-Id: I77cc57d560ff8ac61a3a6bfa1e67efbed6850d09

Setting null context when screen is off

Change-Id: Ibe6605a572e809427e15516b6e824e7abf2cec8e

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

app_tracker: Fix to null context handling

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

Change-Id: I61f617c87b934b203148ac01197f801d2bdcb870

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

Support null context

Null context allows the Screen Reader to yield control to applications
whose existence it is not aware of, like ones using unknown toolkits.
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

Check display state before rebuild navigation context.

After screen off, there's still a sound of context switching.

Change-Id: I4f0b8540fa0e678e8323d27220b93b541554fc6e

Make notification reading nondiscardable

Change-Id: Ic3fa3d1130dab64fa1a85e960bd2be3b3be1c2c2
(cherry picked from commit 145157e5e4cefd19fbd1fbb645e2635b87111929)

screen_reader: check NULL when g_object_ref/unref are called.

Sometimes g_boject_ref/unref are called with NULL object.

Change-Id: Ia9ca8b341fc78a9481f201f29d936bdb15100b07

Make granularity feature optional

Change-Id: If358afacbdc427440201c3316503ccfae6749fa2

Fix handling boolean VConf keys

Types 'bool' and 'int' are distinct in VConf, so boolean keys cannot be read
with vconf_get_int() etc. This patch adds the necessary APIs for 'bool',
alongside 'int' and 'str'.

Change-Id: Ib9b8db0848e6918a1f0bf880f32bf7cae7baf2bb

Revert "Handle Quickpanel"

This reverts commit 31f35ad8364e85f0c04d1a07060dfe762665f2b6.

- Remove the legacy code because DALi app started to support AT-SPI
 from Tizen 7.0.
- Now, screen reader can handle the gestrue even on DALi apps.

Change-Id: Ie18ae6613022e891cb1d083ae19f63c100713638

window_tracker: Keep the D-Bus filter minimal

Do not re-enter D-Bus from inside the D-Bus filter.

Change-Id: Ib8f0757e295c0f7eb79dbce7e8cf930fbd43099c

Add -fno-omit-frame-pointer

This option has a negligible performance impact but it gives an
opportunity to produce more meaningful stacktraces.

Change-Id: I2f1fbce233dc60c3c9db20c5254f78ab3d237c6d

Remember the last touch position for HighlightedObjectInfo

The previous calculation method was to always use the center point of a given
actor, regardless of the position of the touch event that caused it to be
focused. In the tricky scenario where a smaller actor is overlaid on top of a
bigger actor such that the smaller actor covers the center of the bigger actor,
a double-tap-and-hold gesture meant to activate the bigger actor would activate
the smaller actor instead.

The calculation method introduced by this patch is a little bit better, but
still not ideal. However, the HighlightedObjectInfo D-Bus method has only two
parameters, the coordinates, which are then routed back into the app as a touch
event, so there isn't much more that could be done to address this issue.

The logic is ported from: https://review.tizen.org/gerrit/#/c/platform/core/uifw/dali-toolkit/+/288516/

Change-Id: Ia99d48bf1f952975cf55e79f5973fb45de5a306e

Fix memory management issues

Change-Id: Ic276eb376e9d5246c3f366c6c57449e47cfce840

[TV] make the handler of state-changed:checked event aware of live-region politeness

Change-Id: Ia045ddf5c80662b84db1b3dbce8be97df9f364b8

Unification of accessible name composition

Change-Id: I679ef1f8b8f406e6ecccf84bfda167be8d8c1010

[TV] Enable reading on accessible-name change of focused object

Change-Id: I300061fe1e66ac2e656e71e30616bd26394b5b04

Correct AtspiTable usage

Somewhat confusingly, atspi_table_cell_get_table() returns an
AtspiAccessible, not an AtspiTable. The AtspiAccessible can be
then passed to atspi_accessible_get_table_iface() to obtain an
AtspiTable. Also, due to the increased complexity, use `goto`
to clean up all objects at the end.

Change-Id: I912d6b1fc252a52c0d7c9f29c9376d3f036634a1

Add missing #include directives

This fixes the following warnings due to undeclared functions:

src/screen_reader_spi.c:183:6: warning: implicit declaration of function 'isupper' [-Wimplicit-function-declaration]

src/screen_reader_spi.c:185:24: warning: implicit declaration of function 'symbol_lookup' [-Wimplicit-function-declaration]

src/screen_reader_spi.c:185:22: warning: assignment to 'const gchar *' {aka 'const char *'} from 'int' makes pointer from integer without a cast [-Wint-conversion]

src/screen_reader_spi.c:236:23: warning: assignment to 'const gchar *' {aka 'const char *'} from 'int' makes pointer from integer without a cast [-Wint-conversion]

Change-Id: Ib403fd0ae7cc27861e651777e6ef639e56d1bcbb

Fix checking integer values by pointer

Change-Id: Iece746f0cce647510791e31f5ca446f0266c6eb0

Add reading of table cell position (row, column)

This patch adds reading of currently highlighted cell position in table
depending on position of previously highlighted cell. Following rules
apply inclusively:

* announce row if:
    * table has changed (highlight moves from one table to another)
    * row has changed (highlight moves from one row to another row)
    * table cell has not changed (repeated highlight on the same cell)

* announce column if:
    * table has changed (highlight moves from one table to another)
    * column has changed (highlight moves from one column to another column)
    * table cell has not changed (repeated highlight on the same cell)

Also, reading of the currently highlighted cell position in table may be
extended with optional reading of the characteristic of entire table
(table caption and total number of rows and columns).

Presence of this optional reading is controlled by vconf key:

    "db/setting/accessibility/screen_reader/list_grid_information"

The same rules of announcing row and column apply to announcing total rows
and total columns.

Dependency:
https://review.tizen.org/gerrit/#/c/platform/upstream/at-spi2-core/+/293439/

Change-Id: I167c581a942a2416b51f3a52c146a739993a49bd

Force assertive politeness policy for highlighted/focused object

Change-Id: I1d15b32a2eeb5e12d7bec30a16e00ec6f24c74b7

Add reading of usage hint for Slider and SpinButton

Change-Id: I87d7f7b1324d8122cffad8182c5ffb64b063860d

Enable reading of value carried by object with ATSPI_ROLE_SPIN_BUTTON as object's state

Change-Id: I2465f23b296db00719127606777bcdbe43e26aec

Unification of handling value-change event

This patch changes:

* now in all cases when text to read is composed for value-change reading request
  an reading info type customization for STATE reading is taken into consideration;
  previously it was inconsistent
* now TV and Mobile versions use the same logic for handling value-change
  reading request (except for the difference in testing if event's source object
  is currently tracked object, for mobile HIGHLIGHTED state is checked since for TV
  FOCUSED state is checked)
* limitation of handling value-change reading requests in mobile version
  to role SLIDER and FILLER was removed, thans to that chage of page in pagination
  UI control can be anounced by screen reader
* reading an accessible name for FILLER case was removed; since new general
  mechanism for providing textual representation of current value is available
  it should be used (see: https://review.tizen.org/gerrit/#/c/platform/core/accessibility/screen-reader/+/291276/)

Change-Id: I1a809f78332fc155474ee0fe82e83f74ccf9076a

[TV] Apply live region politeness attribute when handling value-changed event

This patch:

* extends applicability of "container-live" ATSPI attribute
  (mapped from WAI-ARIA "aria-live") to case of handling value-change event

* changes default value of live region politeness attribute from "assertive"
  to "off" for text-changed and value-changed events emitted from
  not currently focused UI element

Change-Id: Iff20c8f9855cd19c5b00e59c1319253a15a1971d

Fix memory leak issue

- Add `wav_path` null check

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

Fix memory leak

Change-Id: I39edc1a78dd60b38c6dedd1c30824dd4a8bddb59

Fix potential null pointer dereference

Change-Id: I4198ee8622eabedeeaedff53dc57d218f10d8441

Improve reading on text-changed and caret-moved events in TV version

This patch introduces following features while handling text-changed and
caret-moved events:

* announce capital letters (e.g read "Capital C" for input text "C")
* announce symbols (e.g read "space" for input text " ")
* prevent reveal fragments or whole of the password text

Change-Id: I17e107c2f1adabf7c5bae734d0d580ac20e19776

Customization of value reading

Change-Id: Ic89b14e514c4aea088da1a2e22698d0a2489087f

Use customized tts voice properties with priority over default platform settings

This change allows to controll following tts voice properties for reading
requests originated by screen-reader:

* language (NULL-auto, ANSI code of locale for supported language "ko_KR")
* voice type (0-auto, 1-male, 2-female, 3-child)
* speed (0-auto, 1-slow, ... , 15-fast)

Customized values are passed to screen-reader through vconf keys:

"db/setting/accessibility/screen_reader/tts_voice"
"db/setting/accessibility/screen_reader/tts_speed"

Note:

tts_voice key should contain ":" separated string concatenating language
and voice type like "ko_KR:2"

two above keys should be registered as internal vconf keys on the platform

Change-Id: I1de9f14228dffbcec85fcf584abc6d4cedd3057e

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

Some optimizations to spi_event_get_text_to_read()

These were originally included in the now-reverted patch
"Check active window in spi_event_get_text_to_read()"
(Ic0a65ed5cc77fb232eba1f50c66379c186ec4bd9).

Change-Id: I96ca0a19fed1ec2906699f72a5530ee5e48ef9c1

Revert "Check active window in spi_event_get_text_to_read()"

This reverts commit 5b9831e9abc2546204fa5a941603fc6efb9f566f.

Change-Id: I279b77399c3ac582bbd52ea38a1e023d68b1dd56

Revert "Extended check for being active window object"

This reverts commit 27ad6f0016c19962614b061c0af3aebc1aaec0d8.

Change-Id: Ib3ae2a4baa730aab3596aa15e918a0d0c81e1564

Move _window_extents_get() to utils

There are a few usages of atspi_accessible_get_component_iface() and
atspi_component_get_extents(), followed by error checking and freeing the
resources, that can all be replaced by a call to object_get_extents()
(previously known as _window_extents_get()).

Change-Id: I38952ccf5868e3e99fc1b9227c3772e5c2c382ee

Perform synchronization of local cache of window information also on window:destroy event

Change-Id: I315a230ec28465763143e6238d89b497218ff495

Do not cache window extents

Because:
a) in multi-window model window geometry may change so we need fresh one
b) so far extents were used to identify window at point x,y but we change to
   resource_id for the purpose of window identification

Change-Id: I32fdf6aa5f4c5c2a4e27660754f0ba5945e0bb1b

Remove signal handling code

The signal handler was problematic because it called non-signal-safe
functions (including stdio functions and backtrace_symbols() which calls
dynamic loader functions), causing undefined behaviour.

After removing the problematic functionality all that remained was a
signal handler that terminates the process with an error status code,
which is essentially equivalent to having no signal handler at all, so
all signal-related functionality was removed altogether.

--
https://man7.org/linux/man-pages/man7/signal-safety.7.html

Change-Id: Id669ba998cdb9f11e1c2443a79e8d8afb58beb75

Extended check for being active window object

This check enables dbus communication from TV screen reader not only with
object from the same process as active window but also with objects from
processes providing UIs embedded in active window.

This change aims to reduce chances of dbus calls from TV screen reader to background
processes that respond very slowly.

Change-Id: I0e1786d4cec4e619ec58eff273570cf5e8e2e89d

Check active window in spi_event_get_text_to_read()

This patch tries to prevent D-Bus communication with background
processes, which get very little CPU time and therefore may block
the Screen Reader for an unspecified amount of time.

Change-Id: Ic0a65ed5cc77fb232eba1f50c66379c186ec4bd9