The previous/next text marker from CharacterOffset should be consistent with VisiblePosition so that node start and end will be included when navigating.
<rdar://problem/24733435>
Created attachment 271720 [details] patch
Comment on attachment 271720 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=271720&action=review > Source/WebCore/accessibility/AXObjectCache.cpp:1866 > + // When VisiblePostion move to next node, it will count the leading line break as "moves" to the next node > Source/WebCore/accessibility/AXObjectCache.h:-199 > - CharacterOffset nextCharacterOffset(const CharacterOffset&, bool ignoreStart = true); it's not clear what ignoreStart/ignorePrevEnd means here. can you add a comment or rename this param? /??
Created attachment 271755 [details] patch
Comment on attachment 271755 [details] patch Clearing flags on attachment: 271755 Committed r196824: <http://trac.webkit.org/changeset/196824>
All reviewed patches have been landed. Closing bug.