920543 - [MediaRecorder TEST] timeslice test case, error output

Status: RESOLVED FIXED

(Core :: Audio/Video: Recording, defect)

Description

[u459114]

test_mediarecorder_record_timeslice.html  
Line 60: is(mediaRecorder.mimeType, expectedMimeType,  
Line 61: 'Mime type in ondataavailable = ' + expectedMimeType);

Suppose it should be
Line 60: is(mediaRecorder.mimeType, expectedMimeType,  
Line 61: 'Mime type in ondataavailable = ' + mediaRecorder.mimeType);

An easy first bug for new hand to understand check in process and be familiar with how to run mochitest.

Comment 1

[Jason Smith [:jsmith]]

To fix this, you'll just need to update line 61 to clarify the wording to say:

'Mime type in ondataavailable should be ' + expectedMimeType

Comment 2

[Randy Lin [:rlin] - Be Mozillian on 2015/01/26]

It's a good one to learn how to use the mochitest.

Comment 3

[Shashank VRSN Sabniveesu]

Attachment: BUG 920543 - Correct the 'Mime type in ondataavailable' test case r=rlin

Comment 4

[Randy Lin [:rlin] - Be Mozillian on 2015/01/26]

Comment on attachment 8445636 [details] [diff] [review]
BUG 920543 - Correct the 'Mime type in ondataavailable' test case r=rlin

Review of attachment 8445636 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks a lot. :) r=me, also provide the try result, then you can set check-in needed.

Comment 5

[Shashank VRSN Sabniveesu]

Thank you!

Can you suggest the TRY string? I can't find any component by name 'Core' (and don't want to waste resources!) at http://trychooser.pub.build.mozilla.org/

Comment 6

[Randy Lin [:rlin] - Be Mozillian on 2015/01/26]

Hi Shashank, 
We usually use the "try: -b do -p all -u all -t none" for the final patch.

Comment 7

[Shashank VRSN Sabniveesu]

https://tbpl.mozilla.org/?tree=Try&rev=1a86817cdea4

Comment 8

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/f0359665c820

Comment 9

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/f0359665c820