1081657 - key.wav and special.wav does not sound different

Status: VERIFIED FIXED

(Firefox OS Graveyard :: Gaia::Keyboard, defect)

Description

[Tim Guan-tin Chien [:timdream] (please needinfo)]

special.wav was landed in bug 1061445. It sounds exactly the same with key.wav. I can confirm that too by opening up the files in Audacity.

The strange thing is, they comes with different file size although the codec/channel/wave is complete the same.

Not sure if there is some kind of mix-up.

BTW we need to trim the silence at the beginning and the end, so the user can hear the sound as soon as we play it in the code.

I am not seeing bug 1061445 reach v2.1 but I do see bug 1046336 in v2.1 branch. If this regression is present in v2.1 we need to back it out as well.

Comment 1

[Tim Guan-tin Chien [:timdream] (please needinfo)]

I confirm the bug by import the two wave file into an Audacity project as two channels, and invert one of the channel, and hit play.

Comment 2

[Patryk Adamczyk [:patryk] UX]

Attachment: Correct Sounds

That is strange. I attached the correct WAVs to this bug.

Comment 3

[Pavel Ivanov [:ivanovpavel][:pivanov] UX]

Attachment: patch for Gaia/master

special.wav sound is the same ... this patch contains only key.wav sound change

Comment 4

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Could you trim the file so we don't have a silence before and after the sound?

Comment 5

[Patryk Adamczyk [:patryk] UX]

Attachment: Trimmed Correct Sounds

Attached at the trimmed sounds.

Comment 6

[Pavel Ivanov [:ivanovpavel][:pivanov] UX]

Comment on attachment 8505320 [details]
patch for Gaia/master

Thanks Patryk :)

Hey Tim, it's ready for r?

Comment 7

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Comment on attachment 8505320 [details]
patch for Gaia/master

Assuming you drop the right files.

I am seeing 6 samples (~0.2ms) of silence in the beginning of special.wav from comment 5. Let's ignore it ...

Comment 8

[Pavel Ivanov [:ivanovpavel][:pivanov] UX]

Thanks :)

Landed to master:
https://github.com/mozilla-b2g/gaia/commit/758427b158ce698bc4c211a9262557dd0dc250dd

Comment 9

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Blocking 2.2+ for all fixed regressions.

Comment 10

[Brogan Zumwalt [Inactive]]

While attempting to verify bug, the triggers for key and special sound files appear to have been reversed from 2.1 to 2.2

In 2.1 the higher pitched file is key.wav and the lower pitched file is special.opus

In 2.2 the lower pitched file is key.wav and the higher pitched file is special.wav

Was this a deliberate design decision, or is this an unintentional change?

Besides the above the issue is verified fixed, but I don't want to change status until the above question is resolved.

Comment 11

[Pavel Ivanov [:ivanovpavel][:pivanov] UX]

not sure ... maybe Patryk can help us?

Comment 12

[Patryk Adamczyk [:patryk] UX]

Yes :BroganZ the higher pitched file is correct for regular keys.

Comment 13

[sarsenyev]

Verified the issue is fixed on Flame 2.2

key.wav and special.wav are sound differently 
As per the attached file, higher pitched sound is for special.wav and lower pitched sound is for key.wav on 2.2 Flame

BuildID: 20141209040203
Gaia: 9e0b96c7b61c7ff943876ca93e2596d972437b80
Gecko: acf5660d2048
Gonk: 48835395daa6a49b281db62c50805bd6ca24077e
Version: 37.0a1 (2.2 Master)
Firmware: V188-1
User Agent: Mozilla/5.0 (Mobile; rv:37.0) Gecko/37.0 Firefox/37.0