Bug 101696 - interface with no uuid shouldn't parse. Patch adds a warning, does not prohibit it, and cleans up interfaces that are missing id's r=rginda, sr=jband

mailnews/absync/public/nsIAbSync.idl

@@ -38,6 +38,7 @@
 #include "nsIAbSyncListener.idl"
 #include "nsIMsgStatusFeedback.idl"
 
+[uuid(3c9cbef8-070b-4123-98b2-9297443a8cd1)]
 interface nsIAbSyncState
 { 
   const long nsIAbSyncIdle = 0;

mailnews/addrbook/public/nsIAbBooleanExpression.idl

@@ -75,6 +75,7 @@ typedef long nsAbBooleanOperationType;
  * should be performed between two boolean expressions
  *
  */
+ [uuid(9bdd2e51-2be4-49a4-a558-36d1a812231a)]
 interface nsIAbBooleanOperationTypes
 {
     const long AND        = 0;

mailnews/addrbook/public/nsIAddbookUrl.idl

@@ -53,6 +53,7 @@ interface nsIAddbookUrlOperation
     const long ImportTitle = 8; 
 };
 
+[uuid(ee1d5cc3-5f8c-43f3-b1b8-8d2ea67ce2d8)]
 interface nsIAddbookUrl : nsIURI {
 
 	/////////////////////////////////////////////////////////////////////////////// 

xpcom/base/nsILoggingService.idl

@@ -67,6 +67,7 @@ interface nsILogEventSink : nsISupports
 
 [ptr] native FILE(FILE);
 
+[uuid(ef373116-a9d7-4b94-a3ae-2bc4e10213b3)]
 interface nsIFileLogEventSink : nsILogEventSink
 {
     /**

xpcom/typelib/xpidl/xpidl_header.c

@@ -205,9 +205,9 @@ interface(TreeState *state)
                 id.m3[4], id.m3[5], id.m3[6], id.m3[7]);
         fputc('\n', state->file);
     } else {
-        XPIDL_WARNING((iface, IDL_WARNING1,
-                       "interface %s lacks a uuid attribute",
-                       className));
+        IDL_tree_error(state->tree, "interface %s lacks a uuid attribute\n", 
+            className);
+        FAIL;
     }
 
     if (doc_comments != NULL)