Notify on range stuff even in XUL. Bug 308382, r=sicking, sr=jst

content/base/src/nsGenericElement.cpp

@@ -2747,8 +2747,7 @@ doInsertChildAt(nsIContent* aKid, PRUint32 aIndex, PRBool aNotify,
     }
   }
 
-    // XXX this screws up ranges in XUL, no?  Need to figure out...
-  if (aParent && !aParent->IsContentOfType(nsIContent::eXUL)) {
+  if (aParent) {
     nsRange::OwnerChildInserted(aParent, aIndex);
   }
   

content/xul/content/src/nsXULElement.cpp

@@ -161,6 +161,7 @@
 #include "nsIFrame.h"
 #include "nsNodeInfoManager.h"
 #include "nsXBLBinding.h"
+#include "nsRange.h"
 
 /**
  * Three bits are used for XUL Element's lazy state.
@@ -1120,8 +1121,8 @@ nsXULElement::RemoveChildAt(PRUint32 aIndex, PRBool aNotify)
       }
     }
 
+    nsRange::OwnerChildRemoved(this, aIndex, oldKid);
     mAttrsAndChildren.RemoveChildAt(aIndex);
-    //nsRange::OwnerChildRemoved(this, aIndex, oldKid);
     if (aNotify && doc) {
         doc->ContentRemoved(this, oldKid, aIndex);
     }