[webkit-reviews] review granted: [Bug 129167] After copy and paste, cursor may appear to be above the bottom of content : [Attachment 224900] Patch

[bugzilla-daemon at webkit.org]

Ryosuke Niwa <rniwa at webkit.org> has granted Myles C. Maxfield
<mmaxfield at apple.com>'s request for review:
Bug 129167: After copy and paste, cursor may appear to be above the bottom of
content
https://bugs.webkit.org/show_bug.cgi?id=129167

Attachment 224900: Patch
https://bugs.webkit.org/attachment.cgi?id=224900&action=review

------- Additional Comments from Ryosuke Niwa <rniwa at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=224900&action=review


> Source/WebCore/editing/markup.cpp:632
>      if (accumulator.needRelativeStyleWrapper() &&
needsPositionStyleConversion) {
> +	   accumulator.appendString("<div style=\"clear: both;\"></div>");

Can we add a flag in StylizedAccumulator to look for floats and only generate
this when we saw a float?