Re: [mediacapture-record] Not clear if onpause/onresume should not be fired if it is already paused/recording

Allowing only one "paused" event fired makes sense to me too.
> the check for the paused state must be inside the queued task.

I don't know much about implementation detail but I'm wondering if it is trivial/common practice? As mentioned in #123, for now Firefox and Chrome change `state` synchronously, [so preventing multiple "paused" event is as easy as adding a single if statement...for now](https://cs.chromium.org/chromium/src/third_party/blink/renderer/modules/mediarecorder/media_recorder.cc?l=278).

Currently Chrome does not fire multiple "paused" event, and Firefox (as my fix to add `onpause`/`onresume` is just accepted by the Firefox devs) fires multiple "paused"/"resume" events. Aside from the state transition being async, if W3C prefers only one "pause" event fired I can submit to patch to Firefox.

-- 
GitHub Notification of comment by kbumsik
Please view or discuss this issue at https://github.com/w3c/mediacapture-record/issues/150#issuecomment-428297183 using your GitHub account

Received on Tuesday, 9 October 2018 18:20:32 UTC