[Webkit-unassigned] [Bug 39801] REGRESSION (r45962): Pasted newlines are stripped on form submit at paste.nerv.fi

Sun Jun 6 21:45:12 PDT 2010

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

Shinichiro Hamaji <hamaji at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |darin at apple.com

--- Comment #15 from Shinichiro Hamaji <hamaji at chromium.org>  2010-06-06 21:45:11 PST ---
Thanks for comments, Ojan!

(In reply to comment #14)
> (In reply to comment #12)
> > We might be able to fix this issue by modifying createFragmentFromText so it doesn't emit <div> anymore and uses <br> instead. I didn't choose this way because I guessed we may emit <div> in future for another purpose.
> 
> I think I would prefer this approach for plain text controls. For rich-text, we'd want to maintain the current behavior.
> 
> I would like it even more if we added an ASSERT to text and textWithHardLineBreaks that the text control has no divs in it. It will keep programming for text controls much simpler if we can assume text controls only contain text nodes and BRs.

OK, it seems we were actually using <br> long time ago and Darin made a change so we are using <div> now.

http://trac.webkit.org/changeset/8229

I couldn't see the reason of the change, but I'm sure there were some reasons :) Darin, could you tell us why you made this change?

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