Changeset 30213 in webkit

Timestamp:

Feb 13, 2008 2:00:09 PM (16 years ago)

Author:

justin.garcia@apple.com

Message:

WebCore:

Reviewed by Adam Roben.

<rdar://problem/5729680> REGRESSION (r27873): Removing the last character of a word in Mail or Safari also removes the following space

• editing/Editor.cpp: (WebCore::Editor::deleteWithDirection): Fixed a typo.

LayoutTests:

Reviewed by Adam Roben.

<rdar://problem/5729680> REGRESSION (r27873): Removing the last character of a word in Mail or Safari also removes the following space

• editing/deleting/5729680-expected.txt: Added.
• editing/deleting/5729680.html: Added.

Location:

trunk

Files:

• LayoutTests/ChangeLog (modified) (1 diff)
• LayoutTests/editing/deleting/5729680-expected.txt (added)
• LayoutTests/editing/deleting/5729680.html (added)
• WebCore/ChangeLog (modified) (1 diff)
• WebCore/editing/Editor.cpp (modified) (1 diff)

trunk/LayoutTests/ChangeLog

@@ +1 @@
+2008-02-13  Justin Garcia  <justin.garcia@apple.com>
+
+        Reviewed by Adam Roben.
+        
+        <rdar://problem/5729680> REGRESSION (r27873): Removing the last character of a word in Mail or Safari also removes the following space
+
+        * editing/deleting/5729680-expected.txt: Added.
+        * editing/deleting/5729680.html: Added.
+
 2008-02-13  Nikolas Zimmermann  <zimmermann@kde.org>
 

trunk/WebCore/ChangeLog

@@ +1 @@
+2008-02-13  Justin Garcia  <justin.garcia@apple.com>
+
+        Reviewed by Adam Roben.
+
+        <rdar://problem/5729680> REGRESSION (r27873): Removing the last character of a word in Mail or Safari also removes the following space
+        
+        * editing/Editor.cpp:
+        (WebCore::Editor::deleteWithDirection): Fixed a typo.
+
 2008-02-13  Nikolas Zimmermann  <zimmermann@kde.org>
 

trunk/WebCore/editing/Editor.cpp

@@ -227 +227 @@
         if (isTypingAction) {
             if (m_frame->document()) {
-                TypingCommand::deleteKeyPressed(m_frame->document(), true, granularity);
+                TypingCommand::deleteKeyPressed(m_frame->document(), canSmartCopyOrDelete(), granularity);
                 revealSelectionAfterEditingOperation();
             }