[Webkit-unassigned] [Bug 167116] innerText should replace existing text node

[bugzilla-daemon at webkit.org]

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

Chris Dumez <cdumez at apple.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
           Assignee|webkit-unassigned at lists.web |cdumez at apple.com
                   |kit.org                     |

--- Comment #3 from Chris Dumez <cdumez at apple.com> ---
Created attachment 299312
  --> https://bugs.webkit.org/attachment.cgi?id=299312&action=review
WIP Patch

WIP patch which unfortunately causes a couple of Accessibility tests to fail. I have trouble understanding why but the reason is that we no longer seem to post a AXLiveRegionChanged notification when innerText is set. I believe that previously, those tests were relying on the optimization that was re-using the existing text node and we were posting a AXLiveRegionChanged that the RenderText's text was changed. However, with this patch, we no longer have this optimization and we always remove the existing text node then create a new one. For some reason, this does not seem to post a AXLiveRegionChanged and I do not know if this is expected or not (or why).

-- 
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/20170120/1fb5927e/attachment.html>