Fix for bug 223751 (documentType's nodeValue should be null). r/sr=jst.

content/base/src/nsCommentNode.cpp

@@ -127,6 +127,18 @@ nsCommentNode::GetNodeName(nsAString& aNodeName)
   return NS_OK;
 }
 
+NS_IMETHODIMP
+nsCommentNode::GetNodeValue(nsAString& aNodeValue)
+{
+  return nsGenericDOMDataNode::GetNodeValue(aNodeValue);
+}
+
+NS_IMETHODIMP
+nsCommentNode::SetNodeValue(const nsAString& aNodeValue)
+{
+  return nsGenericDOMDataNode::SetNodeValue(aNodeValue);
+}
+
 NS_IMETHODIMP
 nsCommentNode::GetNodeType(PRUint16* aNodeType)
 {

content/base/src/nsDOMDocumentType.cpp

@@ -180,6 +180,20 @@ nsDOMDocumentType::GetNodeName(nsAString& aNodeName)
   return NS_OK;
 }
 
+NS_IMETHODIMP
+nsDOMDocumentType::GetNodeValue(nsAString& aNodeValue)
+{
+  SetDOMStringToNull(aNodeValue);
+
+  return NS_OK;
+}
+
+NS_IMETHODIMP
+nsDOMDocumentType::SetNodeValue(const nsAString& aNodeValue)
+{
+  return NS_OK;
+}
+
 NS_IMETHODIMP
 nsDOMDocumentType::GetNodeType(PRUint16* aNodeType)
 {

content/base/src/nsGenericDOMDataNode.h

@@ -324,12 +324,8 @@ private:
   NS_IMETHOD GetLocalName(nsAString& aLocalName) {                          \
     return nsGenericDOMDataNode::GetLocalName(aLocalName);                  \
   }                                                                         \
-  NS_IMETHOD GetNodeValue(nsAString& aNodeValue) {                          \
-    return nsGenericDOMDataNode::GetNodeValue(aNodeValue);                  \
-  }                                                                         \
-  NS_IMETHOD SetNodeValue(const nsAString& aNodeValue) {                    \
-    return nsGenericDOMDataNode::SetNodeValue(aNodeValue);                  \
-  }                                                                         \
+  NS_IMETHOD GetNodeValue(nsAString& aNodeValue);                           \
+  NS_IMETHOD SetNodeValue(const nsAString& aNodeValue);                     \
   NS_IMETHOD GetNodeType(PRUint16* aNodeType);                              \
   NS_IMETHOD GetParentNode(nsIDOMNode** aParentNode) {                      \
     return nsGenericDOMDataNode::GetParentNode(aParentNode);                \

content/base/src/nsTextNode.cpp

@@ -120,6 +120,18 @@ nsTextNode::GetNodeName(nsAString& aNodeName)
   return NS_OK;
 }
 
+NS_IMETHODIMP
+nsTextNode::GetNodeValue(nsAString& aNodeValue)
+{
+  return nsGenericDOMDataNode::GetNodeValue(aNodeValue);
+}
+
+NS_IMETHODIMP
+nsTextNode::SetNodeValue(const nsAString& aNodeValue)
+{
+  return nsGenericDOMDataNode::SetNodeValue(aNodeValue);
+}
+
 NS_IMETHODIMP
 nsTextNode::GetNodeType(PRUint16* aNodeType)
 {

content/xml/content/src/nsXMLCDATASection.cpp

@@ -134,6 +134,18 @@ nsXMLCDATASection::GetNodeName(nsAString& aNodeName)
   return NS_OK;
 }
 
+NS_IMETHODIMP
+nsXMLCDATASection::GetNodeValue(nsAString& aNodeValue)
+{
+  return nsGenericDOMDataNode::GetNodeValue(aNodeValue);
+}
+
+NS_IMETHODIMP
+nsXMLCDATASection::SetNodeValue(const nsAString& aNodeValue)
+{
+  return nsGenericDOMDataNode::SetNodeValue(aNodeValue);
+}
+
 NS_IMETHODIMP
 nsXMLCDATASection::GetNodeType(PRUint16* aNodeType)
 {

content/xml/content/src/nsXMLProcessingInstruction.cpp

@@ -133,6 +133,18 @@ nsXMLProcessingInstruction::GetNodeName(nsAString& aNodeName)
   return NS_OK;
 }
 
+NS_IMETHODIMP
+nsXMLProcessingInstruction::GetNodeValue(nsAString& aNodeValue)
+{
+  return nsGenericDOMDataNode::GetNodeValue(aNodeValue);
+}
+
+NS_IMETHODIMP
+nsXMLProcessingInstruction::SetNodeValue(const nsAString& aNodeValue)
+{
+  return nsGenericDOMDataNode::SetNodeValue(aNodeValue);
+}
+
 NS_IMETHODIMP
 nsXMLProcessingInstruction::GetNodeType(PRUint16* aNodeType)
 {