1046391 - [B2G][Flame][2.0][Web Notifications API] "onclose fired" shown on the logcat when notifications are cleared.

Status: RESOLVED INVALID

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

Description

[Yeojin Chung [:YeojinC]]

Attachment: logcat_20140730_OncloseShown.txt

Description:
When the user clears all notifications generated by the notifications API, logcat displays onclose fire.

Repro Steps:
1) Updated Flame to Build ID: 20140730000206.
2) Open Browser, and go to http://mozilla.github.io/qa-testcase-data/webapi/notifications/index.html.
3) Select "Request" under "Request Permission".
4) Under "Show Notification", create 3 different notifications.
5) Pull down the notification overlay, and select "Clear all".

Actual:
"in notification.onclose: onclose fired on title:[...]" is displayed for each notification on logcat.

Expected:
onclose fire is not displayed on logcat.

Flame 2.0 (319mb)

Environmental Variables:
Device: Flame 2.0
Build ID: 20140730000206
Gaia: 2e85678de2c8e13e585288d4cec7d6673cee17ee
Gecko: 6e37ecf873da
Version: 32.0 (2.0)
Firmware Version: v122
User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0


Repro Frequency: 100%
Link to failed test case: https://moztrap.mozilla.org/manage/case/9946/
See attached: logcat

Comment 1

[Yeojin Chung [:YeojinC]]

This issue also reproduces on Flame 2.0(512mb), Flame 2.1(319mb), Flame 1.4(319mb), Buri 2.0, Buri 2.1, Buri 1.4:

Flame 2.0(512mb)

Environmental Variables:
Device: Flame 2.0
Build ID: 20140730000206
Gaia: 2e85678de2c8e13e585288d4cec7d6673cee17ee
Gecko: 6e37ecf873da
Version: 32.0 (2.0)
Firmware Version: v122
User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0

Flame 2.1(319mb)

Environmental Variables:
Device: Flame Master
Build ID: 20140730040205
Gaia: 25e998814ba89f30fe44cd2fdfbb44d160a04641
Gecko: 55c4d770f88b
Version: 34.0a1 (Master)
Firmware Version: v122
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0

Flame 1.4(319mb)

Environmental Variables:
Device: Flame 1.4
Build ID: 20140730063005
Gaia: 142953262d2cb031e3db217206edc3507580b0df
Gecko: 17de7b9d035c
Version: 30.0 (1.4)
Firmware Version: v122
User Agent: Mozilla/5.0 (Mobile; rv:30.0) Gecko/30.0 Firefox/30.0

Buri 2.0

Environmental Variables:
Device: Buri 2.0
Build ID: 20140730063007
Gaia: 2e85678de2c8e13e585288d4cec7d6673cee17ee
Gecko: 2b6108c94b2e
Version: 32.0 (2.0)
Firmware Version: v1.2device.cfg
User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0

Buri 2.1

Environmental Variables:
Device: Buri Master
Build ID: 20140730073003
Gaia: 25e998814ba89f30fe44cd2fdfbb44d160a04641
Gecko: 08c23f12a43e
Version: 34.0a1 (Master)
Firmware Version: v1.2device.cfg
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0

Buri 1.4

Environmental Variables:
Device: Buri 1.4
Build ID: 20140730063005
Gaia: 142953262d2cb031e3db217206edc3507580b0df
Gecko: 17de7b9d035c
Version: 30.0 (1.4)
Firmware Version: v1.2device.cfg
User Agent: Mozilla/5.0 (Mobile; rv:30.0) Gecko/30.0 Firefox/30.0

Onclose fire is displayed for each notification on logcat.

Comment 2

[Michael Henretty [:mikehenrty][:mhenretty]]

This is done on purpose. The clear all button manually closes (and fires the associated event, for each notification). I think we have an invalid test case.

Comment 3

[Jason Smith [:jsmith]]

Kevin - Can you update the test case here to say that onclose should fire?

Comment 4

[KTucker [:KTucker][Inactive 3/4/2016]]

Test case has been updated.

Comment 5

[John Thomas [:Johnt]]

This test case is still valid. Kevin has previously updated the test case of 7/31/14

Comment 6

[KTucker [:KTucker][Inactive 3/4/2016]]

Test case has been updated to state that "onclose should fire":

https://moztrap.mozilla.org/manage/case/9946/