Bug 16661 – REGRESSION (r27369): Paste text into a contenteditable div creates a contenteditable div for each line

• Component: HTML Editing
• Status: RESOLVED FIXED

[Peer Bremer]

Since this weeks webkit builds pasting plain text form a text editor such as bbedit with line endings \n
Creates a new content editable div for each line.

Example pasting:
Test
Test
Test

into a div with contenteditable set to true now creates this inside the div:

<div contenteditable="true">
<div contenteditable="true">Test</div><div contenteditable="true"><br class="webkit-block-placeholder"></div><div contenteditable="true">Test</div><div contenteditable="true"><br class="webkit-block-placeholder"></div><div contenteditable="true">Test</div>
</div>

instead of the desired:

<div contenteditable="true">
<div>Test</div><div><br class="webkit-block-placeholder"></div><div>Test</div><div><br class="webkit-block-placeholder"></div><div>Test</div>
</div>

[David Kilzer (:ddkilzer)]

Confirmed with a local debug build of WebKit r29032 with Safari 3.0.4 (523.12.2) on Mac OS X 10.4.11 (8S165).

[David Kilzer (:ddkilzer)]

The bisect-builds script reports:
Works: r27337  Fails: r27373

[David Kilzer (:ddkilzer)]

The internal autospade script reports:
Works: r27366  Fails: r27369

The only likely candidate is r27369:
http://trac.webkit.org/projects/webkit/changeset/27369

[David Kilzer (:ddkilzer)]

Created attachment 18179 [details]
Test case

* STEPS TO REPRODUCE
1. Launch Safari/WebKit.
2. Open test case (this attachment).
3. Launch BBEdit.
4. Type "Test" followed by the Enter key three times in a new BBEdit document.
5. Select all text.
6. Copy (Cmd-C).
7. Switch to Safari/WebKit.
8. Click in contenteditable div (above the "Test" button).
9. Paste (Cmd-V).
10. Click the "Test" button.

* EXPECTED RESULTS
There should only be one <div> with contenteditable="true".

* ACTUAL RESULTS
Each new line with "Test" text is a <div> with contenteditable="true".

* REGRESSION
This is a regression from shipping Safari 3.0.4 on Mac OS X 10.4.11.

[David Kilzer (:ddkilzer)]

(In reply to comment #4)
> * ACTUAL RESULTS
> Each new line with "Test" text is a <div> with contenteditable="true".

This copies the "id" attribute as well of the parent <div>, which is also incorrect.

[David Kilzer (:ddkilzer)]

<rdar://problem/5665299>

[Justin Garcia]

http://trac.webkit.org/projects/webkit/changeset/31932