[webkit-changes] cvs commit: WebCore/khtml/editing markup.cpp

[Adele]

adele       05/08/23 17:49:05

  Modified:    .        Tag: Safari-2-0-branch ChangeLog
               khtml/editing Tag: Safari-2-0-branch markup.cpp
  Log:
          Merged fix from TOT to Safari-2-0-branch
  
      2005-08-22  David Harrison  <harrison at 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.
  
  Revision  Changes    Path
  No                   revision
  
  
  No                   revision
  
  
  1.1.2.4   +18 -0     WebCore/ChangeLog
  
  Index: ChangeLog
  ===================================================================
  RCS file: /cvs/root/WebCore/ChangeLog,v
  retrieving revision 1.1.2.3
  retrieving revision 1.1.2.4
  diff -u -r1.1.2.3 -r1.1.2.4
  --- ChangeLog	24 Aug 2005 00:17:49 -0000	1.1.2.3
  +++ ChangeLog	24 Aug 2005 00:49:04 -0000	1.1.2.4
  @@ -1,6 +1,24 @@
   2005-08-23  Adele Peterson  <adele at apple.com>
   
           Merged fix from TOT to Safari-2-0-branch
  +
  +    2005-08-22  David Harrison  <harrison at 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-23  Adele Peterson  <adele at apple.com>
  +
  +        Merged fix from TOT to Safari-2-0-branch
           <rdar://problem/4220776> memory leak in KWQArrayImpl::resize() at bugzilla.opendarwin.org
   
       2005-08-17  Beth Dakin  <bdakin at apple.com>
  
  
  
  No                   revision
  
  
  No                   revision
  
  
  1.17.8.2  +1 -1      WebCore/khtml/editing/markup.cpp
  
  Index: markup.cpp
  ===================================================================
  RCS file: /cvs/root/WebCore/khtml/editing/markup.cpp,v
  retrieving revision 1.17.8.1
  retrieving revision 1.17.8.2
  diff -u -r1.17.8.1 -r1.17.8.2
  --- markup.cpp	5 Aug 2005 18:01:20 -0000	1.17.8.1
  +++ markup.cpp	24 Aug 2005 00:49:05 -0000	1.17.8.2
  @@ -592,8 +592,8 @@
                   element->setAttribute(ATTR_CLASS, AppleInterchangeNewline);            
               } else {
                   element = createDefaultParagraphElement(document);
  -                createParagraphContentsFromString(document, element, s);
                   element->ref();
  +                createParagraphContentsFromString(document, element, s);
               }
               fragment->appendChild(element, exceptionCode);
               ASSERT(exceptionCode == 0);