LayoutTests:
authorweinig <weinig@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 24 Feb 2007 02:55:24 +0000 (02:55 +0000)
committerweinig <weinig@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 24 Feb 2007 02:55:24 +0000 (02:55 +0000)
        Reviewed by Maciej.

        - updated test result for http://bugs.webkit.org/show_bug.cgi?id=12856
          REGRESSION (r13685): Canceling the suggestion popup does not revert the change

        * editing/deleting/delete-3800834-fix-expected.txt:

WebCore:

        Reviewed by Maciej.

        - fix http://bugs.webkit.org/show_bug.cgi?id=12856
          REGRESSION (r13685): Canceling the suggestion popup does not revert the change

        * editing/ReplaceSelectionCommand.cpp:
        (WebCore::ReplaceSelectionCommand::doApply): If the fragment to be inserted
        is empty, first delete the selection, and only then bail out.

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

LayoutTests/ChangeLog
LayoutTests/editing/deleting/delete-3800834-fix-expected.txt
WebCore/ChangeLog
WebCore/editing/ReplaceSelectionCommand.cpp

index 3676b13..236fe79 100644 (file)
@@ -1,5 +1,14 @@
 2007-02-23  Mitz Pettel  <mitz@webkit.org>
 
+        Reviewed by Maciej.
+
+        - updated test result for http://bugs.webkit.org/show_bug.cgi?id=12856
+          REGRESSION (r13685): Canceling the suggestion popup does not revert the change
+
+        * editing/deleting/delete-3800834-fix-expected.txt:
+
+2007-02-23  Mitz Pettel  <mitz@webkit.org>
+
         Reviewed by Darin.
 
         - test for http://bugs.webkit.org/show_bug.cgi?id=12823
index f1a3a00..caaaedd 100644 (file)
@@ -38,5 +38,4 @@ layer at (0,0) size 800x600
         RenderBlock (anonymous) at (14,42) size 756x0
         RenderBlock (anonymous) at (14,42) size 756x0
           RenderInline {SPAN} at (0,0) size 0x0
-          RenderText {#text} at (0,0) size 0x0
 caret: position 3 of child 0 {#text} of child 1 {SPAN} of child 1 {DIV} of child 1 {BODY} of child 0 {HTML} of document
index a655b2d..5f84f27 100644 (file)
@@ -1,3 +1,14 @@
+2007-02-23  Mitz Pettel  <mitz@webkit.org>
+
+        Reviewed by Maciej.
+
+        - fix http://bugs.webkit.org/show_bug.cgi?id=12856
+          REGRESSION (r13685): Canceling the suggestion popup does not revert the change
+
+        * editing/ReplaceSelectionCommand.cpp:
+        (WebCore::ReplaceSelectionCommand::doApply): If the fragment to be inserted
+        is empty, first delete the selection, and only then bail out. 
+
 2007-02-23  Anders Carlsson  <acarlsson@apple.com>
 
         Reviewed by Adam.
index 9a05f87..6cde900 100644 (file)
@@ -479,9 +479,6 @@ void ReplaceSelectionCommand::doApply()
     Element* currentRoot = selection.rootEditableElement();
     ReplacementFragment fragment(document(), m_documentFragment.get(), m_matchStyle, selection);
     
-    if (fragment.isEmpty())
-        return;
-    
     if (m_matchStyle)
         m_insertionStyle = styleAtPosition(selection.start());
     
@@ -574,7 +571,7 @@ void ReplaceSelectionCommand::doApply()
     setTypingStyle(0);    
     
     // We're finished if there is nothing to add.
-    if (!fragment.firstChild())
+    if (fragment.isEmpty() || !fragment.firstChild())
         return;
     
     // 1) Insert the content.