966463 - [B2G][Homescreen] Cancelling an app download from Homescreen will result in a grey icon that has no function after a restart

Status: VERIFIED FIXED

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

Description

[Tony Nguyen]

Description:
Downloading an app from the Marketplace will allow the user to return to the Homescreen and cancel the download. After cancelling the download from the homescreen and doing a restart, the app will remain on the homescreen with a grey icon. The icon cannot be removed and nothing happens when tapping on it.

Repro Steps:
1) Updated to BuildID: 20140131004001
2) Turn on Cellular Data and navigate to Marketplace 
3) Search for 'Game Pack' and tap on "Free" to install
4) Immediately press home button
5) Swipe left to navigate to E.me screen
6) Tap and hold on the 'Game Pack' 
7) Tap on the 'x' icon 
8) Tap Delete button
9) Hold the power button and Restart device
10) Upon boot up, move slider to right side to unlock device
11) Swipe left to navigate to E.me screen
12) Tap on grey icon

Actual Result:
Grey icon has no function and user is unable to remove it

Expected Result:
User is able to remove icons from device that are installed from the Marketplace

Environmental Variables:
Device: Buri v1.3 mozRIL
BuildID: 20140131004001
Gaia: 0ddcd8da5bfe1b48c73502ef29220e92f2db6b73
Gecko: 32e45047b663
Version: 28.0a2
v1.2-device.cfg

Repro Rate: 100% (5/5)
See Attached: video

Comment 1

[Jason Smith [:jsmith]]

Pretty sure this is a regression. Can we check if this happens on 1.1?

Comment 2

[pfield]

This issue does not occur on Buri V1.1

When the app is deleted, while downloading, the icon is immediately removed and upon restart of the device, no grey icon is present.

Environmental Variables:
Device: Buri 1.1 Moz
BuildID: 20140130041201
Gaia: c434fe9a0e823029796805e141cfa983cda2d246
Gecko: 1b24a4178317
Version: 18.0
Firmware Version: V1.2-device.cfg

Comment 3

[Cristian Rodriguez (:crdlc)]

Attachment: Patch v1

Simple patch, we had to remove listeners when removing an app from grid too

Comment 4

[pfield]

Removing regressionwindow-wanted tag as it appears significant progress has been made into a resolution towards this bug. (Patch v1 - respectively.)

If a regression window is still wanted, please re-attach tag.

Comment 5

[Cristian Rodriguez (:crdlc)]

Merged in master:

https://github.com/crdlc/gaia/commit/d37ed431d91818ad5f3f91cf8af938c951bef038

Thanks Vivien!

Comment 6

[Jason Smith [:jsmith]]

lgtm on trunk

Comment 7

[John Ford [:jhford] CET/CEST Berlin Time]

Uplifted d37ed431d91818ad5f3f91cf8af938c951bef038 to:
v1.3: 6ac2667bb4b546121844db483e482d0d5f3d7bda