Reviewed by John
authorkocienda <kocienda@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 20 Jan 2005 00:01:58 +0000 (00:01 +0000)
committerkocienda <kocienda@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 20 Jan 2005 00:01:58 +0000 (00:01 +0000)
        * khtml/editing/htmlediting.cpp:
        (khtml::InsertParagraphSeparatorCommand::applyStyleAfterInsertion): Small, cosmetic change John and I decided
        to do on my last checkin, but I forgot to do before landing.

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

WebCore/ChangeLog-2005-08-23
WebCore/khtml/editing/htmlediting.cpp

index fc1f8e9..e86be15 100644 (file)
@@ -2,6 +2,14 @@
 
         Reviewed by John
 
+        * khtml/editing/htmlediting.cpp:
+        (khtml::InsertParagraphSeparatorCommand::applyStyleAfterInsertion): Small, cosmetic change John and I decided
+        to do on my last checkin, but I forgot to do before landing.
+
+2005-01-19  Ken Kocienda  <kocienda@apple.com>
+
+        Reviewed by John
+
         Fix for this bug:
         
         <rdar://problem/3959727> REGRESSION (Mail): Style not preserved on blank lines
index 196958c..d168ec2 100644 (file)
@@ -2596,11 +2596,7 @@ void InsertParagraphSeparatorCommand::applyStyleAfterInsertion()
 
     CSSComputedStyleDeclarationImpl endingStyle(endingSelection().start().node());
     endingStyle.diff(m_style);
-    if (!m_style->length()) {
-        m_style->deref();
-        m_style = 0;
-    }
-    else {
+    if (m_style->length() > 0) {
         applyStyle(m_style);
     }
 }