--- a/testing/xpcshell/runxpcshelltests.py
+++ b/testing/xpcshell/runxpcshelltests.py
@@ -458,20 +458,24 @@ class XPCShellTests(object):
         # It appears most tools expect the time attribute to be present.
         testcase.setAttribute("time", "0")
 
       if "failure" in result:
         failure = doc.createElement("failure")
         failure.setAttribute("type", str(result["failure"]["type"]))
         failure.setAttribute("message", result["failure"]["message"])
 
-        # Lossy translation but required to not break CDATA.
+        # Lossy translation but required to not break CDATA. Also, text could
+        # be None and Python 2.5's minidom doesn't accept None. Later versions
+        # do, however.
         cdata = result["failure"]["text"]
-        if cdata is not None:
-            cdata = cdata.replace("]]>", "]] >")
+        if not isinstance(cdata, str):
+            cdata = ""
+
+        cdata = cdata.replace("]]>", "]] >")
         text = doc.createCDATASection(cdata)
         failure.appendChild(text)
         testcase.appendChild(failure)
 
       if result.get("skipped", None):
         e = doc.createElement("skipped")
         testcase.appendChild(e)