If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

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

RESOLVED FIXED in Firefox 21

Status

()

Core
DOM: Apps
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: jsmith, Assigned: fabrice)

Tracking

Trunk
mozilla21
ARM
Gonk (Firefox OS)
Points:
---
Dependency tree / graph

Firefox Tracking Flags

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

Details

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
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.
(Reporter)

Updated

5 years ago
Blocks: 821288
(Reporter)

Updated

5 years ago
Blocks: 802574
(Assignee)

Comment 1

5 years ago
Created attachment 705258 [details] [diff] [review]
patch
Assignee: nobody → fabrice
Attachment #705258 - Flags: review?(ferjmoreno)
I would have fixed gaia instead for this, but well.
(Assignee)

Comment 3

5 years ago
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.

Updated

5 years ago
Attachment #705258 - Flags: review?(ferjmoreno) → review+
(Assignee)

Comment 5

5 years ago
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+
(Assignee)

Comment 6

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/de3a78d90afa
(Assignee)

Comment 7

5 years ago
https://hg.mozilla.org/releases/mozilla-b2g18/rev/63f6c3b4930a
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
(Reporter)

Updated

5 years ago
Keywords: verifyme
QA Contact: jsmith
(Reporter)

Updated

5 years ago
Keywords: verifyme
https://hg.mozilla.org/mozilla-central/rev/de3a78d90afa
status-b2g18: --- → fixed
status-firefox19: --- → wontfix
status-firefox20: --- → wontfix
status-firefox21: --- → fixed
Target Milestone: --- → mozilla21
You need to log in before you can comment on or make changes to this bug.