1073792 - Audio sound pitch is wrong

Status: VERIFIED FIXED

(Core :: Audio/Video, defect)

Description

[Alice0775 White]

[Tracking Requested - why for this release]: regression

Steps To Reproduce:
1. Open http://www.kulturradio.de/_testfolder_fuer_alle/5-1.html
2. Play back Teaser

Actual Results:
Women voice pitch is too low

Actual Results:
Women voice pitch should be normal. IE11 and Chrome work properly

Comment 1

[Alice0775 White]

Regression window(m-i)
Good:
https://hg.mozilla.org/integration/mozilla-inbound/rev/b91deae84856
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:34.0) Gecko/20100101 Firefox/34.0 ID:20140825183854
Bad:
https://hg.mozilla.org/integration/mozilla-inbound/rev/efe5844c9a5b
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:34.0) Gecko/20100101 Firefox/34.0 ID:20140825193622
Pushlog:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=b91deae84856&tochange=efe5844c9a5b

Regressed by:
efe5844c9a5b	Chris Pearce — Bug 1057879 - Enable MP4Reader on Windows. r=kentuckyfriedtakahe

Comment 2

[Lawrence Mandel [:lmandel] (use needinfo)]

I've got to admit that I'm unless the issue is with the volume of the woman's voice, I'm not sure what the difference is between the browsers. Chris, Anthony - Alice has wonderfully provided a regression window. Can you please review the problem description and see if you can reproduce the issue?

Comment 3

[IU]

(In reply to Lawrence Mandel [:lmandel] from comment #2)
> I've got to admit that I'm unless the issue is with the volume of the
> woman's voice, I'm not sure what the difference is between the browsers.
> Chris, Anthony - Alice has wonderfully provided a regression window. Can you
> please review the problem description and see if you can reproduce the issue?

Basically, it's a speed issue.  It's playing back really really slow -- she sounds like a man with a serious speech impediment.

Comment 4

[Chris Pearce [:cpearce (Not reading bugmail)]]

I can reproduce this bug. We can revert https://hg.mozilla.org/mozilla-central/rev/efe5844c9a5b if we don't get round to fixing this by the time it hits beta.

Comment 5

[Chris Pearce [:cpearce (Not reading bugmail)]]

Attachment: Patch

Pretty easy fix; the WMF MFT doesn't send us a MF_E_TRANSFORM_STREAM_CHANGE for this file when the first output sample has a different stream type than we asked for. So just re-evaluate the output type whenever there's a discontinuity. This will happen on the first sample output, and when we seek.

Comment 6

[Chris Pearce [:cpearce (Not reading bugmail)]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/0f3397b2fff3

We'll want this uplifted to aurora.

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/0f3397b2fff3

Comment 8

[Chris Pearce [:cpearce (Not reading bugmail)]]

Comment on attachment 8497118 [details] [diff] [review]
Patch

Approval Request Comment
[Feature/regressing bug #]: bug 1055383, new MP4 playback backend on Windows
[User impact if declined]: Some video files will not play audio
[Describe test coverage new/current, TBPL]: Did not regress existing mochitests..
[Risks and why]: Low risk.
[String/UUID change made/needed]: None.

Comment 9

[Lawrence Mandel [:lmandel] (use needinfo)]

Comment on attachment 8497118 [details] [diff] [review]
Patch

Aurora+

Comment 10

[Chris Pearce [:cpearce (Not reading bugmail)]]

Aurora checkin needed.

Comment 11

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/c19814f20996

Comment 12

[Alona]

testday-20141107 VERIFIED FIXED

Comment 13

[[SV Manager] Florin Mezei, QA (:FlorinMezei)]

(In reply to Alona from comment #12)
> testday-20141107 VERIFIED FIXED

Alona, on what version was this verified? Was it 34 or 35?

Comment 14

[Alona]

version 34.0. I followed the instructions of Testday.