823431 - [Clock] No ringtone and vibration when alarm goes off from screen off mode.

Status: VERIFIED FIXED

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

Description

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

## Environment :
Unagi phone, release build 2012-12-18
Build info: 
gaia master : 7304395ff0cdfe4de6eb4dd8efc53e89ddca1ed2

## Repro :
1. Set an alarm
2. Turn the screen off
2. Wait for alarm goes off

## Expected:
* It should have ringtone and vibration when alarm goes off.

## Actual:
* There is no ringtone and vibration when alarm goes off.(Alarm actvie screen is showing only.)

Comment 1

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

We check the value of mozHidden during onring's init state. It will let onring page to know that the incoming call happens before the alarm or not(Bug 817485). For the issue, mozHidden is keeping true in init state. It looks like a regression from Bug 823418.

Comment 2

[Joe Cheng [:jcheng] (please needinfo)]

Triage: BB+, C3, P3 - severe usability issue

Comment 3

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

Tim,
I propose to listen 'localized' event for doing init().
Then, the mozHidden is false in init state.
It will also fix Bug 819244 for the l10n warning message.

Comment 4

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

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

Pointer to Github pull-request

Comment 5

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

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

Listening "localized" event to replace setTimeout checking mozHidden in init() state.

Comment 6

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

(In reply to Ian Liu [:ianliu] from comment #3)
> Tim,
> I propose to listen 'localized' event for doing init().
> Then, the mozHidden is false in init state.
> It will also fix Bug 819244 for the l10n warning message.

See, I told you this is unrelated to bug 823418.

setTimeout(fn, 0) is not enough, heh? Don't like the approach but if it works...

Comment 7

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

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

r=me with nit addressed.

Comment 8

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

Since the pr is merged, we can close the issue now.

Comment 9

[John Shih]

verified on unagi
build info
12/27
gaia master: a698a8b4018d5e98008ae4ca91bf0ad49c5591ad
b2g-18: 2008afe879e6