[Webkit-unassigned] [Bug 52268] Setting outerText should convert CR/LF to <br>

[bugzilla-daemon at webkit.org]

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


Emil A Eklund <eae at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #78769|                            |review?
               Flag|                            |




--- Comment #9 from Emil A Eklund <eae at chromium.org>  2011-01-12 18:26:35 PST ---
Created an attachment (id=78769)
 --> (https://bugs.webkit.org/attachment.cgi?id=78769&action=review)
Patch

Rewrote the textToFragment method and made all the changes you suggested except for your last comment about making textToFragment return a text node for strings without a line break. The textToFragment method is used by setInnerText which requires a fragment. I didn't want to make this patch any larger by changing that but I'd be happy to make that change in a later patch if you think it's worthwhile.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.