This is a follow-up of bug 1080470. Pause/suspend and play/resume have different meaning in MediaDecoder. However, for RTSP controller stack, suspend is equal to pause and resume is equal to play. The current implementation of RTSP controller's suspend and resume functions call its pause and play, respectively. We should remove suspend and resume from the IPDL of RTSP controller to avoid confusion and hence improve maintainability.
Hi Vincent and Benjamin, AFAIK, suspend/resume in RTSP controller are equal to pause/play. I tend to do the following: 1) Remove suspend/resume methods from PRtspController.ipdl. 2) Remove RecvSuspend/RecvResume() from RtspControllerParent.h/cpp. We can keep suspend/resume() methods in nsIStreamingProtocolController.idl. This IDL was designed to be a general interface of media stream controller, although it's only used in RTSP streaming. Do you have any concern about it?
I don't think we should remove suspend/resume APIs. We could do something better than pause/play to reduce power consumption.
(In reply to Vincent Chang[:vchang] from comment #2) > I don't think we should remove suspend/resume APIs. We could do something > better than pause/play to reduce power consumption. Do you mean that if we are going to suspend state(user press home key or power key), after a while, the network connection should be disconnected? Then resume the rtsp, we should able to recreate the network connection and seek to the right position?
No one is actually working on RTSP for FxOS right now.
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.