Ooops, actually land the fix with this ChangeLog entry:
authormjs <mjs@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 1 Apr 2006 20:59:45 +0000 (20:59 +0000)
committermjs <mjs@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 1 Apr 2006 20:59:45 +0000 (20:59 +0000)
2006-03-31  Maciej Stachowiak  <mjs@apple.com>

        Reviewed by Adele.

        <rdar://problem/4497684> REGRESSION(NativeTextField): After undoing pasted text in a field, the field changes to only a few pixels in height (8096)

        * editing/ReplaceSelectionCommand.cpp:
        (WebCore::ReplaceSelectionCommand::doApply): merge into start block when pasting into
        an empty editable subtree.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13628 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebCore/editing/ReplaceSelectionCommand.cpp

index afd25fc..b6cbfcd 100644 (file)
@@ -518,9 +518,8 @@ void ReplaceSelectionCommand::doApply()
     bool mergeStart = false;
     if (startBlock == startBlock->rootEditableElement() && startAtStartOfBlock && startAtEndOfBlock) {
         // empty editable subtree, need to mergeStart so that fragment ends up
-        // inside the editable subtree rather than just before it
-        // FIXME: Reconcile comment versus mergeStart = false
-        mergeStart = false;
+        // merged into the editable subtree rather than adding more levels of block nesting
+        mergeStart = true;
     } else {
         // merge if current selection starts inside a paragraph, or there is only one block and no interchange newline to add
         mergeStart = !m_fragment.hasInterchangeNewlineAtStart() &&