708556 - Use a nsISmsRequest object instead of using processId in WebSMS

Status: RESOLVED INVALID

(Core :: DOM: Core & HTML, defect)

Description

[Mounir Lamouri (:mounir)]

Original comment from cjones is in bug 674725 comment 154.

Important part is:
I think all we needed to do was instead of passing around requestIds,
pass around nsISmsRequest references (or something like that).  So the
workflow across processes might have looked something like,

  content process:
    var req = navigator.mozSms.send(...)
      create nsIDOMSmsRequest for |req|.  nsIDOMSmsRequest implements
      nsISmsRequest

      nsISmsService.send(..., req)
         nsRemoteSmsService: create PSmsRequestChild actor under PSms
         that wraps |req|.

  chrome process:
      recv ctor for PSmsRequest.  Create nsISmsRequest that wraps
      PSmsRequestParent.  PSmsRequestParent implements nsISmsRequest.

      nsISmsService.send(..., req)
        ...