badr Reporter
	Description • 6 years ago

User Agent: Mozilla/5.0 (Windows NT 6.1; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/63.0.3239.84 Safari/537.36

Steps to reproduce:

I've called the following methods after I started the mediaRecorder  :
mediaRecorder.pause();
then :
mediaRecorder.resume();

[ Please, check out the full code in the attachments ]


Actual results:

When I call the mediaRecorder.pause() method, the mediaRecorder is actually paused but it doesn't fire the pause event ( onpause ), and the same thing for resume.


Expected results:

It should call the mediaRecorder.onpause() and  mediaRecorder.onresume() event handlers .

	Bumsik Kim Assignee
	Comment 1 • 6 years ago

Hi, I noticed this while working on Bug 1496383. After I looked at the source code (dom/media/MediaRecorder.cpp) I realized it never DispatchSimpleEvent("pause") and DispatchSimpleEvent("resume"). I would like to work on a patch. Could anyone assign me to upload a patch?

	Bumsik Kim Assignee
	Comment 2 • 6 years ago

This resolves the two problems of MediaRecorder:

1. MediaRecorder does not fire pause/resume events when the corresponding methods are called, as mentioned in D7910.
2. The WebIDL for MediaRecorder does not specify onpause/onresume event handler attributes neither.

DispatchSimpleEvent() is used because there are no event attributes needed.

	Bumsik Kim Assignee
	Comment 3 • 6 years ago

The patch to add pause/resume events is just accepted. As I'm not assignee for now I cannot add "checkin-needed" keyword here. Could anyone add "checkin-needed" or make the bug patch landed? Thanks!

	Pulsebot
	Comment 4 • 6 years ago

Pushed by rgurzau@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/8f0aa4187619
Add pause and resume events for MediaRecorder. r=bzbarsky

	Bogdan Tara[:bogdan_tara | bogdant]
	Comment 5 • 6 years ago

Backed out changeset 8f0aa4187619 (Bug 1458538) for idlharness.window.html failures.

Push with failures: https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=8f0aa41876197fc701ddadb73eee8edaa753af19

Backout link: https://hg.mozilla.org/integration/mozilla-inbound/rev/3854b434407792df13fad738667b042e2ebb0449

Failure log: https://treeherder.mozilla.org/logviewer.html#?job_id=204367845&repo=mozilla-inbound&lineNumber=4272

[task 2018-10-09T23:54:05.556Z] 23:54:05     INFO - TEST-START | /mediacapture-record/idlharness.window.html
[task 2018-10-09T23:54:06.183Z] 23:54:06     INFO - 
[task 2018-10-09T23:54:06.183Z] 23:54:06     INFO - TEST-PASS | /mediacapture-record/idlharness.window.html | MediaRecorder interface object name 
[task 2018-10-09T23:54:06.184Z] 23:54:06     INFO - TEST-PASS | /mediacapture-record/idlharness.window.html | MediaRecorder interface: existence and properties of interface prototype object 
[task 2018-10-09T23:54:06.184Z] 23:54:06     INFO - TEST-PASS | /mediacapture-record/idlharness.window.html | MediaRecorder interface: existence and properties of interface prototype object's "constructor" property 
[task 2018-10-09T23:54:06.184Z] 23:54:06     INFO - TEST-PASS | /mediacapture-record/idlharness.window.html | MediaRecorder interface: existence and properties of interface prototype object's @@unscopables property 
[task 2018-10-09T23:54:06.185Z] 23:54:06     INFO - TEST-PASS | /mediacapture-record/idlharness.window.html | MediaRecorder interface: attribute stream 
[task 2018-10-09T23:54:06.185Z] 23:54:06     INFO - TEST-PASS | /mediacapture-record/idlharness.window.html | MediaRecorder interface: attribute mimeType 
[task 2018-10-09T23:54:06.185Z] 23:54:06     INFO - TEST-PASS | /mediacapture-record/idlharness.window.html | MediaRecorder interface: attribute state 
[task 2018-10-09T23:54:06.186Z] 23:54:06     INFO - TEST-PASS | /mediacapture-record/idlharness.window.html | MediaRecorder interface: attribute onstart 
[task 2018-10-09T23:54:06.187Z] 23:54:06     INFO - TEST-PASS | /mediacapture-record/idlharness.window.html | MediaRecorder interface: attribute onstop 
[task 2018-10-09T23:54:06.188Z] 23:54:06     INFO - TEST-PASS | /mediacapture-record/idlharness.window.html | MediaRecorder interface: attribute ondataavailable 
[task 2018-10-09T23:54:06.188Z] 23:54:06     INFO - TEST-UNEXPECTED-PASS | /mediacapture-record/idlharness.window.html | MediaRecorder interface: attribute onpause - expected FAIL
[task 2018-10-09T23:54:06.188Z] 23:54:06     INFO - TEST-INFO | expected FAIL
[task 2018-10-09T23:54:06.189Z] 23:54:06     INFO - 
[task 2018-10-09T23:54:06.189Z] 23:54:06     INFO - TEST-UNEXPECTED-PASS | /mediacapture-record/idlharness.window.html | MediaRecorder interface: attribute onresume - expected FAIL
[task 2018-10-09T23:54:06.189Z] 23:54:06     INFO - TEST-INFO | expected FAIL
[task 2018-10-09T23:54:06.196Z] 23:54:06     INFO - 
[task 2018-10-09T23:54:06.196Z] 23:54:06     INFO - TEST-PASS | /mediacapture-record/idlharness.window.html | MediaRecorder interface: operation requestData() 
[task 2018-10-09T23:54:06.197Z] 23:54:06     INFO - TEST-PASS | /mediacapture-record/idlharness.window.html | MediaRecorder interface: operation isTypeSupported(DOMString) 
[task 2018-10-09T23:54:06.197Z] 23:54:06     INFO - TEST-PASS | /mediacapture-record/idlharness.window.html | MediaRecorder must be primary interface of [object MediaRecorder] 
[task 2018-10-09T23:54:06.198Z] 23:54:06     INFO - TEST-PASS | /mediacapture-record/idlharness.window.html | Stringification of [object MediaRecorder] 
[task 2018-10-09T23:54:06.198Z] 23:54:06     INFO - TEST-PASS | /mediacapture-record/idlharness.window.html | MediaRecorder interface: [object MediaRecorder] must inherit property "stream" with the proper type 
[task 2018-10-09T23:54:06.198Z] 23:54:06     INFO - TEST-PASS | /mediacapture-record/idlharness.window.html | MediaRecorder interface: [object MediaRecorder] must inherit property "mimeType" with the proper type 
[task 2018-10-09T23:54:06.198Z] 23:54:06     INFO - TEST-PASS | /mediacapture-record/idlharness.window.html | MediaRecorder interface: [object MediaRecorder] must inherit property "state" with the proper type 
[task 2018-10-09T23:54:06.199Z] 23:54:06     INFO - TEST-PASS | /mediacapture-record/idlharness.window.html | MediaRecorder interface: [object MediaRecorder] must inherit property "onstart" with the proper type 
[task 2018-10-09T23:54:06.199Z] 23:54:06     INFO - TEST-PASS | /mediacapture-record/idlharness.window.html | MediaRecorder interface: [object MediaRecorder] must inherit property "onstop" with the proper type 
[task 2018-10-09T23:54:06.199Z] 23:54:06     INFO - TEST-PASS | /mediacapture-record/idlharness.window.html | MediaRecorder interface: [object MediaRecorder] must inherit property "ondataavailable" with the proper type 
[task 2018-10-09T23:54:06.199Z] 23:54:06     INFO - TEST-UNEXPECTED-PASS | /mediacapture-record/idlharness.window.html | MediaRecorder interface: [object MediaRecorder] must inherit property "onpause" with the proper type - expected FAIL
[task 2018-10-09T23:54:06.200Z] 23:54:06     INFO - TEST-INFO | expected FAIL
[task 2018-10-09T23:54:06.200Z] 23:54:06     INFO - 
[task 2018-10-09T23:54:06.200Z] 23:54:06     INFO - TEST-UNEXPECTED-PASS | /mediacapture-record/idlharness.window.html | MediaRecorder interface: [object MediaRecorder] must inherit property "onresume" with the proper type - expected FAIL
[task 2018-10-09T23:54:06.201Z] 23:54:06     INFO - TEST-INFO | expected FAIL
[task 2018-10-09T23:54:06.216Z] 23:54:06     INFO - .................................
[task 2018-10-09T23:54:06.217Z] 23:54:06     INFO - TEST-OK | /mediacapture-record/idlharness.window.html | took 638ms

	Bumsik Kim Assignee
	Comment 6 • 6 years ago

It looks like the failure caused by the updated WebIDL. I'll check it out and push a patch for this.

	Jean-Yves Avenard [:jya]
	Comment 7 • 6 years ago

indeed you need to update the web-platform-test result expectation.

it's located there:
https://searchfox.org/mozilla-central/source/testing/web-platform/meta/mediacapture-record/idlharness.window.js.ini

for tests that used to fail but are now passing, it's just a matter of removing the corresponding entries.

More information in: https://developer.mozilla.org/en-US/docs/Mozilla/QA/web-platform-tests

You can test locally with the command:

./mach web-platform-tests --include=mediacapture-record

you can create this as an extra commit, with a dependency on the previous one.

	Bumsik Kim Assignee
	Comment 8 • 6 years ago

I was about to ask about it. Thanks for the information.

	Boris Zbarsky [:bzbarsky]
	Comment 9 • 6 years ago

Jean-Yves, did you see the question in the Phabricator review?

	Pulsebot
	Comment 10 • 6 years ago

Pushed by jyavenard@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/80b0b8b446fd
Add pause and resume events for MediaRecorder. r=jya,bzbarsky

	Jean-Yves Avenard [:jya]
	Comment 11 • 6 years ago

already pushed. thank you

	Cristina Coroiu [:ccoroiu]
	Comment 12 • 6 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/80b0b8b446fd

	Andreas Pehrson [:pehrsons]
	Comment 14 • 5 years ago
dev-doc-info

This implemented the pause and resume events for MediaRecorder, and the corresponding onpause and onresume handlers. We're fixing the behavior and spec to be more web compatible in bug 1514016 so please take that into account when writing dev docs.

	Chris Mills [:cmills]
	Comment 15 • 5 years ago

(In reply to Andreas Pehrson [:pehrsons] from comment #14)
> This implemented the pause and resume events for MediaRecorder, and the
> corresponding onpause and onresume handlers. We're fixing the behavior and
> spec to be more web compatible in bug 1514016 so please take that into
> account when writing dev docs.

Ah. I think when I originally documented MediaRecorder, I just wrote all the reference pages including pause/onpause and reume/onresume, and assumed that because pause()/resume() worked, they would fire. They are listed as being supported since Firefox 25. That looks not to be the case ;-)

I'll see to it that these details are updated asap next year.

For now, I've added a note to the Fx65 rel notes:
https://developer.mozilla.org/en-US/docs/Mozilla/Firefox/Releases/65#APIs