Last Comment Bug 770297 - Call to isLaunchable in getSelf/getInstalled/getAll/getNotInstalled needs to be the app origin, not the install origin
: Call to isLaunchable in getSelf/getInstalled/getAll/getNotInstalled needs to ...
Product: Firefox Graveyard
Classification: Graveyard
Component: Web Apps (show other bugs)
: unspecified
: All All
: -- normal
: Firefox 16
Assigned To: :Felipe Gomes (needinfo me!)
: Jason Smith [:jsmith]
: 770307 770311 770380 (view as bug list)
Depends on:
  Show dependency treegraph
Reported: 2012-07-02 12:50 PDT by Jason Smith [:jsmith]
Modified: 2016-02-04 15:00 PST (History)
6 users (show)
See Also:
QA Whiteboard:
Iteration: ---
Points: ---

Patch (2.47 KB, patch)
2012-07-02 17:15 PDT, :Felipe Gomes (needinfo me!)
fabrice: review+
Details | Diff | Splinter Review

Description Jason Smith [:jsmith] 2012-07-02 12:50:00 PDT

1. Go to
2. Install a free web application
3. Uninstall the free web application
4. Go to app page for the app in step #2


The app install button should be showed as "install"


The app install button is still showed as "installed." Under the hood, they are using getInstalled() to determine this.
Comment 1 Jason Smith [:jsmith] 2012-07-02 14:15:53 PDT
Looks like this generally applies to getAll() and getInstalled()
Comment 3 Jason Smith [:jsmith] 2012-07-02 17:12:04 PDT
Working with Felipe, we've figured out the root cause. Looks like the calls to isLaunchable in getSelf/getInstalled/getAll/getNotInstalled is using the install origin, not the app origin. That explains why there's a lot of problems being seen.
Comment 4 Jason Smith [:jsmith] 2012-07-02 17:13:24 PDT
*** Bug 770307 has been marked as a duplicate of this bug. ***
Comment 5 Jason Smith [:jsmith] 2012-07-02 17:13:46 PDT
*** Bug 770380 has been marked as a duplicate of this bug. ***
Comment 6 :Felipe Gomes (needinfo me!) 2012-07-02 17:15:14 PDT
Created attachment 638545 [details] [diff] [review]

Huh I was using the wrong origin to check if an app was installed, which had the amusing effect of considering if an app was installed or not based on if you had the Marketplace app itself installed
Comment 7 Jason Smith [:jsmith] 2012-07-02 17:16:44 PDT
*** Bug 770311 has been marked as a duplicate of this bug. ***
Comment 8 :Felipe Gomes (needinfo me!) 2012-07-02 20:48:06 PDT
Comment 9 Jason Smith [:jsmith] 2012-07-02 22:01:13 PDT
Did a good amount of testing with the try build on Windows - it works! Did tests such as:

- Installing and uninstalling apps on marketplace
- Viewing apps on the myapps dashboard
- Installing apps with and without appcache support
- Testing apps natively vs. acquired
Comment 10 Marco Castelluccio [:marco] 2012-07-03 16:01:57 PDT
Just tested with Linux and it's working.
Comment 11 Jason Smith [:jsmith] 2012-07-11 18:27:31 PDT
Verified on Nightly for Windows 7, Windows XP, OS X 10.7, and Ubuntu 12.

Note You need to log in before you can comment on or make changes to this bug.