832273 - [CLOCK] Alarm is not ringing after snoozing the first time

Status: VERIFIED FIXED

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

Description

[Carlos Martínez Toral [:carlosmartinez]]

Tested in unagi with Gecko-c893ed7.Gaia-d381947.

STR:
1-Open clock app
2-Set alarm 
3-Wait for the alarm to ring once
4-Snooze it
5-Wait for the alarm to ring again

Expected result --> It should ring after five minutes by default
Actual result --> The screen is shown after five minutes as it´s ringing but not sound is played

Comment 1

[[e-mail], [e-mail]]

I can reproduce it when alarm goes off sometimes... The root cause is the timing of checking 'document.mozHidden' to start alarm notification. It looks like the localized event coming more early than before. So, we get the value true of 'document.mozHidden' to set a silent alarm.

I can have a quickly fixing by setTimeout with zero delay time(previous solution). But we still need to have a test it meticulously.

Comment 2

[[e-mail], [e-mail]]

Attachment: Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/7697

Pointer to Github pull-request

Comment 3

[[e-mail], [e-mail]]

Comment on attachment 704420 [details]
Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/7697

Tim,
We still need a delay time to check the mozHidden. It's the previous solution. Could you please review the pr?
Thank you.

Comment 4

[[e-mail], [e-mail]]

Since the pr is merged(https://github.com/mozilla-b2g/gaia/pull/7697), we can close the issue now.

Comment 5

[[e-mail], [e-mail]]

Note: Merge in v1.0.0
https://github.com/mozilla-b2g/gaia/pull/7724

Comment 6

[Askeing Yen[:askeing][:fyen]]

Gaia:     6c53dfd2e5d60393f6a57e9e1a4b5af6a3d6a783
Gecko:    http://hg.mozilla.org/releases/mozilla-b2g18/rev/2226f181c8c2
BuildID   20130126070202
Version   18.0

Verified.

Comment 7

[Lukas Blakk [:lsblakk] use ?needinfo]

Landed on mozilla-b2g18/gaia master prior to the 1/25 branching to mozilla-b2g18_v1_0_0/v1.0.0, updating status-b2g-v1.0.0 to fixed.