Patch for proper re-setting of frame selection's typing style.

Source/core/editing/DeleteSelectionCommand.cpp

Index: Source/core/editing/DeleteSelectionCommand.cpp
diff --git a/Source/core/editing/DeleteSelectionCommand.cpp b/Source/core/editing/DeleteSelectionCommand.cpp
index cf7b3be70ed4e148722ad027425de707d9f3dd0d..506a96b3bad7f59a462c53acc0613c8eecb93835 100644
--- a/Source/core/editing/DeleteSelectionCommand.cpp
+++ b/Source/core/editing/DeleteSelectionCommand.cpp
@@ -285,8 +285,10 @@ void DeleteSelectionCommand::saveTypingStyleState()
     // typing style at the start of the selection, nor is there a reason to
     // compute the style at the start of the selection after deletion (see the
     // early return in calculateTypingStyleAfterDelete).

[yosin_UTC9, 2013/09/06 08:10:16]

We should update this comment.

-    if (m_upstreamStart.deprecatedNode() == m_downstreamEnd.deprecatedNode() && m_upstreamStart.deprecatedNode()->isTextNode())
+    if (m_upstreamStart.deprecatedNode() == m_downstreamEnd.deprecatedNode() && m_upstreamStart.deprecatedNode()->isTextNode()) {
+        document()->frame()->selection()->clearTypingStyle();

[yosin_UTC9, 2013/09/06 08:10:16]

nit: It seems you are working on old source code. Frame::selection() now returns
reference instead of pointer of FrameSelection.

[yosin_UTC9, 2013/09/06 08:10:16]

Should we call clearTypingStyle() in calculateTypingStyleAfterDelete()?

void DeleteSelectionCommand::calculateTypingStyleAfterDelete()
{
    if (!m_typingStyle) {
        document()->frame()->selection().clearTypingStyle();
        return;
    }
    ...
}

[arpitab_, 2013/09/06 14:01:32]

My bad! Have made the appropriate change in the new patch-set.

[arpitab_, 2013/09/06 14:01:32]

Yes! Although I don't see much difference between the two by way of
functionality. However, this seems more appropriate and in keeping with what we
do in Editor.cpp as well.
Have moved the clearTypingStyle() call to this place in patch-set 2.

         return;
+    }
 
     if (shouldNotInheritStyleFrom(*m_selectionToDelete.start().anchorNode()))
         return;