Checking in Pete Zha's <pete.zha@sun.com> fix for bug 121726. Don't remove and re-insert form controls into form.elements when the name or id changes. This caused form.elements to be incorrectly ordered and is pointless to start with. r=jkeiser@iname.com, sr=jst@netscape.com

content/html/content/src/nsGenericHTMLElement.cpp

@@ -4170,8 +4170,6 @@ nsGenericHTMLElement::SetFormControlAttribute(nsIForm* aForm,
     if (!tmp.IsEmpty()) {
       aForm->RemoveElementFromTable(thisControl, tmp);
     }
-
-    aForm->RemoveElement(thisControl);
   }
 
   if (aForm && aName == nsHTMLAtoms::type) {
@@ -4198,8 +4196,6 @@ nsGenericHTMLElement::SetFormControlAttribute(nsIForm* aForm,
     if (!tmp.IsEmpty()) {
       aForm->AddElementToTable(thisControl, tmp);
     }
-
-    aForm->AddElement(thisControl);
   }
 
   if (aForm && aName == nsHTMLAtoms::type) {