Fixing bug 181351. Making NS_NewTextNode() return an nsITextContent pointer in stead of reaturning an nsIContent pointer to avoid having to QI the new text node to set it's value. r=caillon@returnzero.com, sr=bzbarsky@mit.edu

content/base/src/nsDOMAttribute.cpp

@@ -289,14 +289,13 @@ nsDOMAttribute::GetFirstChild(nsIDOMNode** aFirstChild)
   }
   if (!value.IsEmpty()) {
     if (!mChild) {      
-      nsIContent* content;
+      nsCOMPtr<nsITextContent> content;
 
-      result = NS_NewTextNode(&content);
+      result = NS_NewTextNode(getter_AddRefs(content));
       if (NS_OK != result) {
         return result;
       }
       result = content->QueryInterface(NS_GET_IID(nsIDOMText), (void**)&mChild);
-      NS_RELEASE(content);
     }
     mChild->SetData(value);
     result = mChild->QueryInterface(NS_GET_IID(nsIDOMNode), (void**)aFirstChild);

content/build/nsContentModule.cpp

@@ -345,7 +345,7 @@ MAKE_CTOR(CreateImageDocument,            nsIDocument,                 NS_NewIma
 MAKE_CTOR(CreateCSSParser,                nsICSSParser,                NS_NewCSSParser)
 MAKE_CTOR(CreateCSSLoader,                nsICSSLoader,                NS_NewCSSLoader)
 MAKE_CTOR(CreateHTMLElementFactory,       nsIElementFactory,           NS_NewHTMLElementFactory)
-MAKE_CTOR(CreateTextNode,                 nsIContent,                  NS_NewTextNode)
+MAKE_CTOR(CreateTextNode,                 nsITextContent,              NS_NewTextNode)
 //MAKE_CTOR(CreateAnonymousElement,         nsIContent,                  NS_NewAnonymousElement)
 MAKE_CTOR(CreateAttributeContent,         nsIContent,                  NS_NewAttributeContent)
 MAKE_CTOR(CreateXMLElementFactory,        nsIElementFactory,           NS_NewXMLElementFactory)