Reviewed by Justin.
authorharrison <harrison@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 22 Aug 2005 20:24:39 +0000 (20:24 +0000)
committerharrison <harrison@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 22 Aug 2005 20:24:39 +0000 (20:24 +0000)
        <rdar://problem/4221384> Denver Regression: stickies widget crashed on 8F15 when pasting text

        Test cases added:  None.  Problem triggered by simple paste, which lots of the
        editing tests already do, but also required particular memory layout/cleanup state, which
        is not specifiable in our tests.

        * khtml/editing/markup.cpp:
        (khtml::createFragmentFromText):
        Ref the new paragraph element right away, rather than allocating more memory first.

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

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

index ac6a28e..96f4721 100644 (file)
@@ -1,3 +1,17 @@
+2005-08-22  David Harrison  <harrison@apple.com>
+
+        Reviewed by Justin.
+
+        <rdar://problem/4221384> Denver Regression: stickies widget crashed on 8F15 when pasting text
+
+        Test cases added:  None.  Problem triggered by simple paste, which lots of the
+        editing tests already do, but also required particular memory layout/cleanup state, which
+        is not specifiable in our tests.
+
+        * khtml/editing/markup.cpp:
+        (khtml::createFragmentFromText):
+        Ref the new paragraph element right away, rather than allocating more memory first.
+
 2005-08-22  Eric Seidel  <eseidel@apple.com>
         Fix by Tobias Lidskog <tobiaslidskog@mac.com>
 
index 5d4edd7..d57acff 100644 (file)
@@ -594,8 +594,8 @@ DOM::DocumentFragmentImpl *createFragmentFromText(DOM::DocumentImpl *document, c
                 element->setAttribute(classAttr, AppleInterchangeNewline);            
             } else {
                 element = createDefaultParagraphElement(document);
-                createParagraphContentsFromString(document, element, s);
                 element->ref();
+                createParagraphContentsFromString(document, element, s);
             }
             fragment->appendChild(element, exceptionCode);
             ASSERT(exceptionCode == 0);