Bug 14792 – REGRESSION: Copy inserts carriage return in middle of selection

• Component: New Bugs
• Status: RESOLVED FIXED

[Cameo Wood]

When copying a url from a facebook page, (html from page is attached) the selection is truncated.

I am trying to copy the url: "http://loicandhugh.eventbrite.com/"

and when I copy it via Safari, it copies instead:
"http://loicandhugh.eventbr
ite.com/" 
with a carriage return in the middle of the url, causing it to appear to be truncated when one pasts it into the address field.

[Cameo Wood]

Created attachment 15710 [details]
source of offending page

I should mention this behaviour does not happen in IE or Firefox

[David Kilzer (:ddkilzer)]

The HTML being copied looks like this:

<span>http://loicandhugh.eventbr</span><wbr></wbr><span class="word_break"></span>ite.com/

This does not occur with Safari 2.0.4 (419.3) with original WebKit on Mac OS X 10.4.10 (8R218), so it's a regression.

[David Kilzer (:ddkilzer)]

<rdar://problem/5367763>

Even simpler:

foo<span style="float:left"></span>bar

Created attachment 15873 [details]
Do not emit newlines around floating or positioned blocks

The change, as well as the updated test results, matches WinIE behavior.

[Justin Garcia]

Mitz what does this do to spellchecking of words at the beginning or end of a floating or positioned block?  As in:

<div contenteditable="true">please<div style="float: right;">test</div>this</div>

Since we no longer emit newlines before and after this block, won't pleasetestthis be passed to the spellchecker?

(In reply to comment #6)
> Since we no longer emit newlines before and after this block, won't
> pleasetestthis be passed to the spellchecker?

That's exactly what happens.

[Justin Garcia]

Comment on attachment 15873 [details]
Do not emit newlines around floating or positioned blocks

So now we'll incorrectly mark 'test' as misspelled whereas before where we'd fail to mark "pleasethis" as being misspelled.  Seems like we're no worse off, r=me.

[Mark Rowe (bdash)]

Landed in r25234.