[webkit-reviews] review requested: [Bug 53389] [Gtk] atk_text_set_caret_offset returns True even when it is unsuccessful : [Attachment 80646] Patch proposal + unit tests

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Jan 31 09:17:24 PST 2011


Mario Sanchez Prada <msanchez at igalia.com> has asked  for review:
Bug 53389: [Gtk] atk_text_set_caret_offset returns True even when it is
unsuccessful
https://bugs.webkit.org/show_bug.cgi?id=53389

Attachment 80646: Patch proposal + unit tests
https://bugs.webkit.org/attachment.cgi?id=80646&action=review

------- Additional Comments from Mario Sanchez Prada <msanchez at igalia.com>
Attaching a patch to fix this issue.

Notice that I included the needed code to consider list item markers (so we
return FALSE in case you try to set the caret in the middle of them), as it's
part of the solution for this bug, even if bug 53388 is not fixed yet and hence
set_caret_offset will return FALSE always regardless of the offset being in the
item's marker or in the item's text :-)

Asking for review now...


More information about the webkit-reviews mailing list