Closed Bug 833659 Opened 8 years ago Closed 8 years ago

REINSTALL_FORBIDDEN error isn't triggering when installing a packaged app with a mini-manifest off of X origin when a hosted app was already installed from X origin

Categories

(Core Graveyard :: DOM: Apps, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(firefox19 wontfix, firefox20 wontfix, firefox21 fixed, b2g18 fixed)

RESOLVED FIXED
mozilla21
Tracking Status
firefox19 --- wontfix
firefox20 --- wontfix
firefox21 --- fixed
b2g18 --- fixed

People

(Reporter: jsmith, Assigned: fabrice)

References

Details

Attachments

(1 file)

Build: B2G 18 1/22/2013
Device: Unagi

Steps:

1. Go to http://mozqa.com/webapi-permissions-tests/
2. Install "Hosted App Test Case 1"
3. Install "Packaged App Test Case 2"

Expected:

App install after step #3 should fail with REINSTALL_FORBIDDEN.

Actual:

App install is successful. This is problematic because when you try uninstall both apps, the packaged app will uninstall, but the hosted app will not uninstall until the phone is restarted (i.e. icon is left behind). Likely due to the single app per origin constraint.
Blocks: 821288
Blocks: app-install
Attached patch patchSplinter Review
Assignee: nobody → fabrice
Attachment #705258 - Flags: review?(ferjmoreno)
I would have fixed gaia instead for this, but well.
Doing gaia workarounds for issues that lead to something corrupted in the backend is not a good idea, and gaia is not the only userr of this api.
In this specific case, the backend is not corrupted... since the reboot fixes this.

This is really the homescreen not handling correctly the uninstall.

But it's maybe easier to just disable this in gecko for now.
Attachment #705258 - Flags: review?(ferjmoreno) → review+
Comment on attachment 705258 [details] [diff] [review]
patch

[Approval Request Comment]
This is part of a set of patches that helps hardening against edge cases that may let the phone in a bad state. Risk is low compared to the reward.
Attachment #705258 - Flags: approval-mozilla-b2g18?
Attachment #705258 - Flags: approval-mozilla-b2g18? → approval-mozilla-b2g18+
https://hg.mozilla.org/releases/mozilla-b2g18/rev/63f6c3b4930a
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Keywords: verifyme
QA Contact: jsmith
Keywords: verifyme
Target Milestone: --- → mozilla21
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.