Enable/fix testcases for MediaRecorder api on Fennec.

RESOLVED FIXED in Firefox 48

Status

()

defect
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: bechen, Assigned: bechen)

Tracking

unspecified
mozilla48
All
Unspecified
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox48 fixed)

Details

Attachments

(3 attachments)

See bug 1215115 comment 22.

I want to enable [test_mediarecorder_record_canvas_captureStream.html] and
[test_mediarecorder_record_changing_video_resolution.html] at all platforms.
And fix test_mediarecorder_webm_support.html since we remove the vorbis encoder.
See Also: → 1234230
Comment on attachment 8740855 [details]
MozReview Request: Bug 1261007 - part2: fix test_mediarecorder_webm_support. r=rillian

https://reviewboard.mozilla.org/r/46047/#review42601

::: dom/media/test/test_mediarecorder_webm_support.html:16
(Diff revision 1)
>  ok(MediaRecorder.isTypeSupported('video/webm'), 'Should support video/webm');
> -ok(MediaRecorder.isTypeSupported('video/webm; codecs="vp8, vorbis"'), 'Should support video/webm + vp8/vorbis');
> +ok(!MediaRecorder.isTypeSupported('video/webm; codecs="vp8, vorbis"'), 'Should not support video/webm + vp8/vorbis');
>  ok(!MediaRecorder.isTypeSupported('video/webm; codecs="vp9, vorbis"'), 'Should not support video/webm + vp9/vorbis');
> +ok(MediaRecorder.isTypeSupported('video/webm; codecs="vp8, opus"'), 'Should support video/webm + vp8/vorbis');
> +ok(!MediaRecorder.isTypeSupported('video/webm; codecs="vp9, opus"'), 'Should not support video/webm + vp9/vorbis');
>  </script>

Message is incorrect for the two new checks. It should be updated to reference opus instead of vorbis. For example:

```
ok(MediaRecorder.isTypeSupported('video/webm; codecs="vp8, opus"'), 'Should support video/webm + vp8/opus');
ok(!MediaRecorder.isTypeSupported('video/webm; codecs="vp9, opus"'), 'Should not support video/webm + vp9/opus');
```
Attachment #8740855 - Flags: review?(giles) → review+
Comment on attachment 8740854 [details]
MozReview Request: Bug 1261007: part1: Force to send video sample into encoder if we got the same video sample more than 1 seconds. Enable testcases. r=jolin

https://reviewboard.mozilla.org/r/46045/#review42855

Now that TrackEncoder sends only unique images, the logic in OmxVideoTrackEncoder::GetEncodedTrack() seems unnecessary and it would be nice to be removed.

::: dom/media/test/mochitest.ini:705
(Diff revision 1)
>  tags=msg
>  [test_mediarecorder_record_audionode.html]
>  tags=msg
>  [test_mediarecorder_record_canvas_captureStream.html]
> -skip-if = (android_version < '17' || toolkit == 'android') # Android/Gonk before SDK version 17 does not have the OMX Encoder API and Fennec does not support video recording
>  tags=msg

This would break B2G but I guess we don't care anymore.

::: dom/media/test/mochitest.ini
(Diff revision 1)
>  tags=msg
>  [test_mediarecorder_record_canvas_captureStream.html]
> -skip-if = (android_version < '17' || toolkit == 'android') # Android/Gonk before SDK version 17 does not have the OMX Encoder API and Fennec does not support video recording
>  tags=msg
>  [test_mediarecorder_record_changing_video_resolution.html]
> -skip-if = android_version < '17' # Android/Gonk before SDK version 17 does not have the OMX Encoder API.

Ditto.
Attachment #8740854 - Flags: review?(jolin)
Comment on attachment 8740854 [details]
MozReview Request: Bug 1261007: part1: Force to send video sample into encoder if we got the same video sample more than 1 seconds. Enable testcases. r=jolin

Forgot to click 'Ship it!'. :$
Attachment #8740854 - Flags: review+
Comment on attachment 8740855 [details]
MozReview Request: Bug 1261007 - part2: fix test_mediarecorder_webm_support. r=rillian

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/46047/diff/1-2/
Comment on attachment 8740854 [details]
MozReview Request: Bug 1261007: part1: Force to send video sample into encoder if we got the same video sample more than 1 seconds. Enable testcases. r=jolin

https://reviewboard.mozilla.org/r/46045/#review43601
Attachment #8740854 - Flags: review+
Comment on attachment 8742209 [details]
MozReview Request: Bug 1261007 - part3: Remove the same/redundant code of checking the unique image. r=jolin

https://reviewboard.mozilla.org/r/46975/#review43603

LGTM. Thanks!
Attachment #8742209 - Flags: review?(jolin) → review+
Keywords: checkin-needed
You need to log in before you can comment on or make changes to this bug.