Bug 53453 – [Gtk] atk_text_get_selection/atk_text_set_selection fails for list items

• Status: RESOLVED
• Resolution: FIXED
• Component: Accessibility

[Mario Sanchez Prada]

Due to the same reason behind bug 53388 and bug 53436, the actual behaviour with  atk_text_get_selection and atk_text_set_selection when applied to list items is not the expected one, hence needs fixing.

Steps to reproduce:

1. Open the test case attached to bug 53388 (attachment 80582 [details]) in Epiphany.

2. Select all the text right after the 'First' word (' item'), in the first list item.

3. Using Accerciser, attempt to get and set the selection within that list item, and check that the actual results, by:

A. Selecting the corresponding object in Accerciser's tree of accessible objects

B1. Typing 'acc.queryText().setSelection(0, 0, 5)'

  Expected outcome: False (as you shouldn't be able to select inside the item's marker)
  Actual outcome: True

B2. Typing 'acc.queryText().setSelection(0, 3, 8)'

  Expected outcome: True (+ you'll see the 'First' substring selected)
  Actual outcome: True (but nothing is selected)

C. Manually select the 'First' substring and type 'acc.queryText().getSelection(0)'

  Expected outcome: (0, 5)
  Actual outcome: (3, 8)

[Mario Sanchez Prada]

Created attachment 81533 [details]
Patch proposal + unit test

Attaching patch proposal + an unit test updated to check this new stuff

[Martin Robinson]

Comment on attachment 81533 [details]
Patch proposal + unit test

View in context: https://bugs.webkit.org/attachment.cgi?id=81533&action=review

> Source/WebCore/accessibility/gtk/AccessibilityObjectWrapperAtk.cpp:1117
> +    int end = endOffset;

Why not just continue to use endOffset?

[Mario Sanchez Prada]

(In reply to comment #2)
> (From update of attachment 81533 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=81533&action=review
> 
> > Source/WebCore/accessibility/gtk/AccessibilityObjectWrapperAtk.cpp:1117
> > +    int end = endOffset;
> 
> Why not just continue to use endOffset?

Because we need to use its value even after having set 'end', which can change its value along the function, thus differing from the original endoffset value.

[Mario Sanchez Prada]

Committed r78978: <http://trac.webkit.org/changeset/78978>

[WebKit Review Bot]

http://trac.webkit.org/changeset/78978 might have broken GTK Linux 32-bit Release
The following tests are not passing:
fast/events/popup-when-select-change.html
fast/forms/listbox-hit-test-zoomed.html
fast/forms/listbox-typeahead-greek.html
fast/forms/select-type-ahead-non-latin.html
fast/spatial-navigation/snav-single-select.html
inspector/debugger/debug-inlined-scripts.html
inspector/debugger/debugger-no-nested-pause.html
inspector/debugger/debugger-step-in.html
platform/gtk/accessibility/combo-box-collapsed-selection-changed.html
platform/gtk/fast/forms/menulist-typeahead-find.html