Bug 344453: AttributeChanged events sent out twice. Also make CharacterDataChanged notifications go out even for orphaned nodes. r/sr=jst

content/base/src/nsGenericDOMDataNode.cpp

@@ -1012,7 +1012,7 @@ nsGenericDOMDataNode::SetText(const PRUnichar* aBuffer,
   }
 
   // Trigger a reflow
-  if (aNotify && document) {
+  if (aNotify) {
     nsNodeUtils::CharacterDataChanged(this, PR_FALSE);
   }
 }

content/xul/document/src/nsXULDocument.cpp

@@ -947,11 +947,6 @@ nsXULDocument::AttributeChanged(nsIDocument* aDocument,
         }
     }
 
-    // Now notify external observers
-    NS_DOCUMENT_NOTIFY_OBSERVERS(AttributeChanged,
-                                 (this, aElement, aNameSpaceID,
-                                  aAttribute, aModType));
-
     // See if there is anything we need to persist in the localstore.
     //
     // XXX Namespace handling broken :-(