One more for the speedracer bustage fix.

The problem is that the generated header contains the following bit:
  enum { MODIFICATION = 1U };
  enum { ADDITION = 2U };
  enum { REMOVAL = 3U };

So to speedracer's compiler, this line signals a potential programmer error:

PRInt32 modHint = modification ? nsIDOMMutationEvent::MODIFICATION : nsIDOMMutationEvent::ADDITION;

Apparently they value it like an error and not a warning.

content/xul/content/src/nsXULElement.cpp

@@ -3086,7 +3086,8 @@ nsXULElement::SetAttr(nsINodeInfo* aNodeInfo,
             (tagName.get() == nsXULAtoms::key))
             return rv;
 
-        PRInt32 modHint = modification ? nsIDOMMutationEvent::MODIFICATION : nsIDOMMutationEvent::ADDITION;
+        PRInt32 modHint = modification ? PRInt32(nsIDOMMutationEvent::MODIFICATION)
+                                       : PRInt32(nsIDOMMutationEvent::ADDITION);
         mDocument->AttributeChanged(this, attrns, attrName, modHint, 
                                     NS_STYLE_HINT_UNKNOWN);
       }