[webkit-reviews] review requested: [Bug 62092] setting innerText to an empty string on editable div loses focus : [Attachment 96786] html test file that demonstrates the problem - browser compatible

[bugzilla-daemon at webkit.org]

Una Sabovic <una.sabovic at palm.com> has asked  for review:
Bug 62092: setting innerText to an empty string on editable div loses focus
https://bugs.webkit.org/show_bug.cgi?id=62092

Attachment 96786: html test file that demonstrates the problem - browser
compatible
https://bugs.webkit.org/attachment.cgi?id=96786&action=review

------- Additional Comments from Una Sabovic <una.sabovic at palm.com>
Attached is the html demo that demonstrates the problem a little further and is
browser compatible. 
There are two test cases. Both test cases work correctly in Firefox. Problem is
present in Chrome & Safari.
With the attached patch webkit qt launcher behaves the same as Firefox.

The code I changed in FrameSelection.cpp had a FIXME which I believe this patch
fixes. FIXME said

// FIXME: When endpoints are removed, we should just alter the selection,
instead of blowing it away.