[Webkit-unassigned] [Bug 30883] [Gtk] Implement AtkText for HTML elements which contain text

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=30883


Joanmarie Diggs <joanmarie.diggs at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #46791|                            |review?
               Flag|                            |




--- Comment #29 from Joanmarie Diggs <joanmarie.diggs at gmail.com>  2010-01-17 23:18:41 PST ---
Created an attachment (id=46791)
 --> (https://bugs.webkit.org/attachment.cgi?id=46791)
Handle the issue from comment #28, add sanity checks to
objectAndOffsetUnignored, new tests, style compliance changes....

This patch makes the following changes:

1. Moves the text collapsing/assembling functionality that was in
getPangoLayoutForAtk to the newly-created textForObject.

2. webkit_accessible_text_get_text now falls back on textForObject when it
comes up empty.

3. Adds a check in textForObject so that we don't double-add newlines when we
have a BR.

4. Adds new tests for items 2 and 3 above to testatk.c

5. (Hopefully) brings the rest of testatk.c into compliance with the style
guidelines. Also there were several instances of our calling
atk_object_ref_accessible_child(), but never unref'ing the object at the end of
the test. I assume we should be unref'ing them.

6. Adds some sanity checks to objectAndOffsetUnignored which (hopefully)
address Xan's concerns.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.