811087 - [sms] No sound is played when receiving a text message with vibration and sound enabled

Status: VERIFIED FIXED

(Firefox OS Graveyard :: Gaia::SMS, defect, P1)

Description

[Marcia Knous [:marcia]]

Bug 807918 is back using today's build - found during the smoketest:

Tested with:

Gecko-11e6372e034f7f19ad0b39749b48425a462120af
Gaia-92b7a950de3ec1cdecd90fb9d2cd7f930bb1f227

STR:
1-In Settings, confirm that ring and vibrate are on, as well as the alerts box is checked. 
2-Send a text message from another phone
3-Receive a text message with vibration and sound enabled

Expected result --> Device vibrates and default sound is played
Actual result --> Device vibrates but no sound is played

Comment 1

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Bug 809680 - Notification should come up with a sound. ?

Though vibration should also be included.

Comment 2

[Marcia Knous [:marcia]]

When I have tested this before, when you get an SMS there was some kind of default system sound that was made - today that is not happening. The specific smoketest that covers this is:

[SMS] Receive a sms with vibration and sound (device unlocked) & confirm notification

So perhaps something changed, but if so we will have to adjust the smoketest to reflect that.

Comment 3

[(inactive after 6/18) Alive Kuo [:alive]]

(In reply to Marcia Knous [:marcia] from comment #2)
> When I have tested this before, when you get an SMS there was some kind of
> default system sound that was made - today that is not happening. The
> specific smoketest that covers this is:
> 
> [SMS] Receive a sms with vibration and sound (device unlocked) & confirm
> notification
> 
> So perhaps something changed, but if so we will have to adjust the smoketest
> to reflect that.

You are right, the sms sound is temporarily removed, and later would be merged into notification sounds.

Comment 4

[Faramarz Rashed]

:alive, is there a reason for temporarily removing this? sounds like this has been done intentionally.

Comment 5

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

I have been told that it will be fixed by bug 801351

Comment 6

[Alex Keybl [:akeybl]]

Marking for C2, given this meets the criteria of known P1/P2 blocking-basecamp+ bugs at the end of C1.

Comment 7

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

https://github.com/mozilla-b2g/gaia/commit/a1ba7afbbaafd81aa2914383f7d3e76df4ceccad

Comment 8

[rafael.marquez]

In today's version the bug it's not resolved
Gecko-04c2d4f
Gaia-1087df8

Comment 9

[Marcia Knous [:marcia]]

Using:

Gaia: 1910ebaae9bda968f7da6b0ed42eef906013af30
Gecko: 72dbb825d86b5d003929c3ee1efe1187abd3646e

I now am getting the sound again.

Comment 10

[Marcia Knous [:marcia]]

Verified fixed using the build in Comment 9.

Comment 11

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

David did you change the the RESOLVED -> REOPENED by mistake?

Comment 12

[:Gavin Sharp [email: [e-mail]]]

Per dev-gaia, I think he was suggesting that this has regressed. I filed bug 819259, since reopening a FIXED bug is generally not a good idea (makes the bug history very confusing).