[Webkit-unassigned] [Bug 159910] AX: Incorrect behavior for word related text marker functions when there's collapsed whitespace

[bugzilla-daemon at webkit.org]

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

--- Comment #7 from chris fleizach <cfleizach at apple.com> ---
Comment on attachment 284009
  --> https://bugs.webkit.org/attachment.cgi?id=284009
patch

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

> Source/WebCore/accessibility/AXObjectCache.cpp:1590
> +        bool cumulateEnough = validateOffset ? cumulativeOffset + lastStartOffset >= offset : cumulativeOffset >= offset;

the name of this variable is strange: cumulateEnough
what purpose do you want it to serve?

it seems like it could be "bool validOffset"

> Source/WebCore/accessibility/AXObjectCache.cpp:2307
> +    // SimplifiedBackwardsTextIterator ignores replaced element

period after this sentence

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20160719/251ed8cc/attachment.html>