Bug 742790 - Part 1/5: support SMS delivery failure event. Interface changes, sr=sicking,mounir

dom/sms/interfaces/nsIDOMSmsManager.idl

@@ -8,7 +8,7 @@ interface nsIDOMEventListener;
 interface nsIDOMMozSmsRequest;
 interface nsIDOMMozSmsFilter;
 
-[scriptable, builtinclass, uuid(be78baf4-20ba-4ceb-be40-d9774a9388c4)]
+[scriptable, builtinclass, uuid(1bee1224-56a2-4935-af7b-0011746306cb)]
 interface nsIDOMMozSmsManager : nsIDOMEventTarget
 {
   unsigned short      getNumberOfMessagesForText(in DOMString text);
@@ -30,5 +30,6 @@ interface nsIDOMMozSmsManager : nsIDOMEventTarget
 
   [implicit_jscontext] attribute jsval onreceived;
   [implicit_jscontext] attribute jsval onsent;
-  [implicit_jscontext] attribute jsval ondelivered;
+  [implicit_jscontext] attribute jsval ondeliverysuccess;
+  [implicit_jscontext] attribute jsval ondeliveryerror;
 };

dom/sms/interfaces/nsIDOMSmsMessage.idl

@@ -4,13 +4,19 @@
 
 #include "nsISupports.idl"
 
-[scriptable, builtinclass, uuid(fc58ba6e-70de-4550-aa1e-790ecc19cf98)]
+[scriptable, builtinclass, uuid(5e3224dd-1ca3-43e6-861e-b7734656fd2f)]
 interface nsIDOMMozSmsMessage : nsISupports
 {
   // TODO: we should add SENT and RECEIVED DOMString constants, see bug 443316.
 
   readonly attribute long      id;
   readonly attribute DOMString delivery;  // Should be "sent" or "received".
+  /**
+   * For received messages, it is "success"; for sent messages, it can be
+   * "pending", "success", "error", or "not-applicable" if the message was sent
+   * without status report requisition.
+   */
+  readonly attribute DOMString deliveryStatus;
   readonly attribute DOMString sender;
   readonly attribute DOMString receiver;
   readonly attribute DOMString body;