Bug 26960

Summary: [Gtk] caret offset not updated when selecting text
Product: WebKit Reporter: Joanmarie Diggs <jdiggs>
Component: AccessibilityAssignee: Nobody <webkit-unassigned>
Status: VERIFIED FIXED    
Severity: Normal CC: apinheiro, jmalonzo, walker.willie, xan.lopez
Priority: P3 Keywords: Gtk
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Linux   
Bug Depends on:    
Bug Blocks: 25531    
Attachments:
Description Flags
caretoffset.patch jmalonzo: review+

Description Joanmarie Diggs 2009-07-03 21:40:24 PDT
Steps to reproduce:

1. Launch Epiphany and Accerciser.

2. In Epiphany, position the caret within some text.

3. In Accerciser's tree of accessibles on the left, highlight the accessible text object which corresponds to the object chosen in step 2.

4. In Accerciser's iPython Console, type:

     acc.queryText().caretOffset

5. In Epiphany, press Shift + Right Arrow two times to select two characters to the right

6. In Accerciser's iPython Console, type:

     acc.queryText().caretOffset

Expected results: The offset returned in step 6 would reflect the new location of the caret (the step 4 offset + 2).

Actual results: The offset returned in step 6 is the same as the step 4 offset.

(Xan, I believe that the fix for this bug, combined with your fixes for bug 26814 and bug 26815, should cause the presentation of selected text by ATs like Orca to JustWork(tm) -- which will be awesome. Sorry for not catching this one the other day!!)
Comment 1 Xan Lopez 2009-07-04 09:03:50 PDT
Created attachment 32255 [details]
caretoffset.patch

Well, it was basically the same bug than the other cases, but in a different place.
Comment 2 Jan Alonzo 2009-07-04 14:20:35 PDT
Comment on attachment 32255 [details]
caretoffset.patch

This looks fine. r=me.

We need to think of a way to test these changes in the future while we don't have an AXController in DRT yet. Maybe more unit tests.
Comment 3 Joanmarie Diggs 2009-07-06 08:18:11 PDT
This patch seems to be working nicely. Thanks!!
Comment 4 Jan Alonzo 2009-07-07 03:51:30 PDT
(In reply to comment #3)
> This patch seems to be working nicely. Thanks!!

Thanks for verifying Joanmarie. Fix landed in r45549.
Comment 5 Joanmarie Diggs 2009-07-27 17:53:04 PDT
(re)Verifying. :-) Thanks.