911901 - [WAP push] If exit wap push message via home key and open another push msg, content will be empty and title become "wap-push-message" and unable to close it.

Status: VERIFIED FIXED

(Firefox OS Graveyard :: Gaia, defect)

Description

[Enpei]

Attachment: video

If exit wap push message via home key and open another push msg, content will be empty and title become "wap-push-message" and unable to close it.

* Build Number                
Gaia:     9fb5802df60a9081846d704def01df814ed8fbd4
Gecko:    http://hg.mozilla.org/mozilla-central/rev/b6c29e434519
BuildID   20130901040215
Version   26.0a1

* Reproduce Steps
1. Send 2 wap push messages to DUT.(using NowSMS)
2. Open 1 message and press home key to exit it.
3. Open another one via drag down notification bar.

* Expected Result
Another content can be viewed.

* Actual Result
1. Content is empty.
2. Title is changed from sender's number to "wap-push-message".

* Occurrence rate
100%

Comment 1

[Enpei]

Attachment: logcat

Comment 2

[Gabriele Svelto [:gsvelto]]

Attachment: [PATCH] Close the WAP Push attention screen when the home button is pressed

This patch detects when the attention screen is shrunk into the status-bar (which happens when the home button is pressed) and closes it instead. The patch also ensures that the entire application closes itself when the attention screen is dismissed.

Comment 3

[Gabriele Svelto [:gsvelto]]

Comment on attachment 802357 [details] [diff] [review]
[PATCH] Close the WAP Push attention screen when the home button is pressed

Awww... wrong bug, this should have been in bug 911895.

Comment 4

[Enpei]

Hi Gabriele,

Is this fixed as well since bug 911895 is fixed now?

Comment 5

[Gabriele Svelto [:gsvelto]]

(In reply to Enpei from comment #4)
> Is this fixed as well since bug 911895 is fixed now?

Not yet. I need a different code-path for this and I will refactor part of the app accordingly but I want to land bug 911247 before that.

Comment 6

[Gabriele Svelto [:gsvelto]]

Quick update on this. Notifications were thoroughly broken so since I had to touch that code already in bug 911247 this will also be fixed as part of it (a patch is available there already that also fixes the issue).

Comment 7

[Enpei]

This is not found on 
Gaia:     313599c293f596519604070fa378ffc89e61e98f
Gecko:    http://hg.mozilla.org/releases/mozilla-aurora/rev/decf58f0d596
BuildID   20131008004009
Version   26.0a2

but found some other bugs can still reproduce the issue.

Comment 8

[Gabriele Svelto [:gsvelto]]

This was fixed as part of bug 911247.

Comment 9

[Enpei]

Verified on Buri.

Gaia:     d0f0110f6a907ef386134dc693f5dc5a632a0e9e
Gecko:    http://hg.mozilla.org/releases/mozilla-aurora/rev/818ef9b4096c
BuildID   20131013004004
Version   26.0a2