1021301 - [B2G][Window Mgmt]There is no sound or vibration when receiving a notification with the screen locked

Status: RESOLVED DUPLICATE of bug 1023642

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

Description

[Andrew Stole [:astole]]

Attachment: logcat

When receiving a notification while the screen is locked, the notification pops up, but there is no sound or vibration. Also, when the notification is received, the wallpaper gets desaturated.

Repro Steps:
1) Update a Flame to BuildID: 20140605040202
2) Make sure sound and vibrate are on and lock the device
3) Send a message to the device

Actual:
The notification displays, but there is no sound or vibration

Expected:
The phone vibrates and there is a sound with the notification

2.0 Environmental Variables:
Device: Flame 2.0 MOZ
BuildID: 20140605040202
Gaia: d2cfef555dabab415085e548ed44c48a99be5c32
Gecko: 51b428be6213
Version: 32.0a1
Firmware Version: v10G-2

Repro frequency: 100%, 3/3
See attached: logcat

Comment 1

[Andrew Stole [:astole]]

This issue does not occur on 1.4. The device vibrates and there is sound with the notification.

1.4 Environmental Variables:
Device: Flame 1.4 MOZ
BuildID: 20140605000203
Gaia: 1f238df7ac68a73a4ceb27391a204c800f38ab1c
Gecko: a7b7f1b579cc
Version: 30.0
Firmware Version: v10G-2

Comment 2

[Duane Dixon [:ddixon]]

B2G Inbound Regression Window

Last Working:

Device: Flame 2.0
Build ID: 20140602233007
Gaia: 5dfe4ec36758b5d9aaa241926be9de2d8c4da7d4
Gecko: 6999490835c7
Version: 32.0a1 (2.0) MOZ
Firmware Version: v10G-2

First Broken

Device: Flame 2.0
Build ID: 20140603023003
Gaia: 969d35114eed0eabbd0fe7c01fad19a2df0097b5
Gecko: 8b3454b9896d
Version: 32.0a1 (2.0) MOZ
Firmware Version: v10G-2


Last Working Gecko and First Broken Gaia
Issue DOES NOT occur. 
Gaia: 969d35114eed0eabbd0fe7c01fad19a2df0097b5
Gecko: 6999490835c7

Last Working Gaia and First Broken Gecko
Issue DOES occur. 
Gaia: 5dfe4ec36758b5d9aaa241926be9de2d8c4da7d4
Gecko: 8b3454b9896d

Gaia Pushlog: 
https://github.com/mozilla-b2g/gaia/compare/5dfe4ec36758b5d9aaa241926be9de2d8c4da7d4...969d35114eed0eabbd0fe7c01fad19a2df0097b5

Comment 3

[Jason Smith [:jsmith]]

Broken by bug 1014142. Can you take a look?

Comment 4

[Raniere Silva]

Sorry but I believe that my patch isn't what cause this regression:

$ git bisect log                                                                  
# bad: [6b5be16c8d1fd78936f05ce786a9cfd2eff38b2e] Merge pull request #20298 from gitmai/bug-1008987-import-vcard
# good: [5dfe4ec36758b5d9aaa241926be9de2d8c4da7d4] Merge pull request #19773 from rnowm/1017509
git bisect start 'HEAD' '5dfe4ec36758b5d9aaa241926be9de2d8c4da7d4'
# bad: [3713e8cb04b2d6f14a9dad3140ebc150a1383f2f] Merge pull request #19780 from guilherme-pg/bug1014816-fmd-fxa-logout
git bisect bad 3713e8cb04b2d6f14a9dad3140ebc150a1383f2f
# bad: [dc5e69bf457750ce4fa3075ea97d48044e6a5278] Merge pull request #20020 from gasolin/issue-1020387
git bisect bad dc5e69bf457750ce4fa3075ea97d48044e6a5278
# bad: [e116dbdf84e91fa55c6387bbfe188840a17fa1fa] Merge pull request #19930 from crdlc/bug-1019509
git bisect bad e116dbdf84e91fa55c6387bbfe188840a17fa1fa
# bad: [4fa4d94260651056020605698f1870047afd7bc4] Merge pull request #19937 from ahal/gaia_unit_mozrunner
git bisect bad 4fa4d94260651056020605698f1870047afd7bc4
# bad: [cdaf998934f4924a4b256ed5458b484818a01948] Merge pull request #19769 from borjasalguero/loop_contacts_integration
git bisect bad cdaf998934f4924a4b256ed5458b484818a01948
# bad: [070f1a8b9ed631c68458cd2321f53c0f100da5ac] Merge pull request #19911 from jmcanterafonseca/fix_merge_photos
git bisect bad 070f1a8b9ed631c68458cd2321f53c0f100da5ac
# good: [2680cf4fc70c59cf801aab8283c0b3178175e3cf] Revert "Bug 960720 - Change the client side Smart Collection wallpapers [r=ranbena]"
git bisect good 2680cf4fc70c59cf801aab8283c0b3178175e3cf
# bad: [7a09fc92d54368e7dcbe7ec76bf2ebf0ff854d8c] Merge pull request #19712 from mnjul/bug_950884_lockscreen_visual_refresh
git bisect bad 7a09fc92d54368e7dcbe7ec76bf2ebf0ff854d8c
# bad: [9b729b38e00bd893cd2f520b7f54e572a10e9c04] Bug 950884 - [VsD Refresh] Lockscreen Visual Refresh
git bisect bad 9b729b38e00bd893cd2f520b7f54e572a10e9c04
# first bad commit: [9b729b38e00bd893cd2f520b7f54e572a10e9c04] Bug 950884 - [VsD Refresh] Lockscreen Visual Refresh

Test at Hamachi since I don't have Fame:

OS version: 2.1.0.0-prerelease
Platform version: 33.0a1
Build Identifier: 20140610213121

Comment 5

[Jason Smith [:jsmith]]

Reflagging regressionwindow to double check the window here.

Comment 6

[Duane Dixon [:ddixon]]

B2G Inbound Regression Window (Buri device)

Last Working

Device: Buri 2.0
Build ID: 20140603024935
Gaia: dd9e74d196675b0b05170f0ab94a80a36697a551
Gecko: a1a83cc4882f
Version: 32.0a1 (2.0) 
Firmware Version: v1.2device.cfg


First Broken

Device: Buri 2.0
Build ID: 20140603030735
Gaia: 070f1a8b9ed631c68458cd2321f53c0f100da5ac
Gecko: 5c13c87bdfce
Version: 32.0a1 (2.0) 
Firmware Version: v1.2device.cfg

Last Working Gecko and First Broken Gaia 
Issue DOES occur here. (this points to a gaia issue)
Gaia: 070f1a8b9ed631c68458cd2321f53c0f100da5ac
Gecko: a1a83cc4882f

Last Working Gaia and First Broken Gecko
Issue DOES NOT occur here. 
Gaia: dd9e74d196675b0b05170f0ab94a80a36697a551
Gecko: 5c13c87bdfce


Gaia Pushlog: 
https://github.com/mozilla-b2g/gaia/compare/dd9e74d196675b0b05170f0ab94a80a36697a551...070f1a8b9ed631c68458cd2321f53c0f100da5ac

Comment 7

[Joshua Mitchell (Inactive)]

Re-worked Regression window with a very 'nice' candidate: Bug 950884

Comment 8

[Jason Smith [:jsmith]]

John - Can you take a look?

Comment 9

[John Lu [:mnjul][Please NI or I'll miss]]

Yes, and good news: Patch for bug 1023642 will also fix this issue. In nutshell, the javascript error prevented sound-playing codes/... to be executed.

Please do note that the "desaturation" of wallpaper is a specified UX/Visual change in bug 950884, though.