[webkit-changes] cvs commit: WebCore/khtml/xml dom_docimpl.cpp

Adele adele at opensource.apple.com
Fri Jul 8 15:44:46 PDT 2005


adele       05/07/08 15:44:46

  Modified:    .        Tag: Ti-2005-007-branch ChangeLog
               khtml/xml Tag: Ti-2005-007-branch dom_docimpl.cpp
  Log:
         Merged for for <rdar://problem/4168001> from TOT to Ti-2005-007-branch.
  
      2005-07-07  Adele Peterson  <adele at apple.com>
  
          change by Maciej, reviewed by me.
  
          fix for <rdar://problem/4168001> Cambridge 8C45: Safari crashes in Webcore (DOM::NodeImpl::removeAllEventListeners() + 32)
  
          No test cases added.  There are some edge cases in which removeAllDisconnectedNodeEventListeners can get
          called twice for the same document, and this can cause some nodes to unnecessarily remain in the
          m_disconnectedNodesWithEventListeners list.  This fix just clears the m_disconnectedNodesWithEventListeners
          list after removing all event listeners for the disconnected nodes.
  
          * khtml/xml/dom_docimpl.cpp: (DocumentImpl::removeAllDisconnectedNodeEventListeners):
  
  Revision  Changes    Path
  No                   revision
  
  
  No                   revision
  
  
  1.4104.2.37.2.1 +17 -0     WebCore/ChangeLog
  
  Index: ChangeLog
  ===================================================================
  RCS file: /cvs/root/WebCore/ChangeLog,v
  retrieving revision 1.4104.2.37
  retrieving revision 1.4104.2.37.2.1
  diff -u -r1.4104.2.37 -r1.4104.2.37.2.1
  --- ChangeLog	28 Jun 2005 00:26:28 -0000	1.4104.2.37
  +++ ChangeLog	8 Jul 2005 22:44:36 -0000	1.4104.2.37.2.1
  @@ -1,3 +1,20 @@
  +2005-07-08  Adele Peterson  <adele at apple.com>
  +
  +       Merged for for <rdar://problem/4168001> from TOT to Ti-2005-007-branch.
  +
  +    2005-07-07  Adele Peterson  <adele at apple.com>
  +
  +        change by Maciej, reviewed by me.
  +
  +        fix for <rdar://problem/4168001> Cambridge 8C45: Safari crashes in Webcore (DOM::NodeImpl::removeAllEventListeners() + 32)
  +
  +        No test cases added.  There are some edge cases in which removeAllDisconnectedNodeEventListeners can get 
  +        called twice for the same document, and this can cause some nodes to unnecessarily remain in the 
  +        m_disconnectedNodesWithEventListeners list.  This fix just clears the m_disconnectedNodesWithEventListeners 
  +        list after removing all event listeners for the disconnected nodes.
  +
  +        * khtml/xml/dom_docimpl.cpp: (DocumentImpl::removeAllDisconnectedNodeEventListeners):
  +
   === WebCore-415.11.1 ===
   
   2005-06-27  Adele Peterson  <adele at apple.com>
  
  
  
  No                   revision
  
  
  No                   revision
  
  
  1.211.6.3.2.1 +1 -0      WebCore/khtml/xml/dom_docimpl.cpp
  
  Index: dom_docimpl.cpp
  ===================================================================
  RCS file: /cvs/root/WebCore/khtml/xml/dom_docimpl.cpp,v
  retrieving revision 1.211.6.3
  retrieving revision 1.211.6.3.2.1
  diff -u -r1.211.6.3 -r1.211.6.3.2.1
  --- dom_docimpl.cpp	3 Jun 2005 21:11:45 -0000	1.211.6.3
  +++ dom_docimpl.cpp	8 Jul 2005 22:44:45 -0000	1.211.6.3.2.1
  @@ -1241,6 +1241,7 @@
            ++iter) {
           iter.current()->removeAllEventListeners();
       }
  +    m_disconnectedNodesWithEventListeners.clear();
   }
   
   #if APPLE_CHANGES
  
  
  



More information about the webkit-changes mailing list