731545 - Implement the web app installation UI

Status: RESOLVED DUPLICATE of bug 697006

(Firefox Graveyard :: Web Apps, defect)

Description

[:Felipe Gomes (needinfo for replies!)]

Implement the notification (bar/doorhanger) and related UI that will be presented when a web app requests to be installed through navigator.mozApps

Comment 1

[:Felipe Gomes (needinfo for replies!)]

Tim, so the way that things are hooked together between the webpage and the install process is:

 a) webpage calls navigator.mozApps.install(..) to install itself
 b) the install function will then send a notification called webapps-ask-install
 c) front-end observes this notification and presents a doorhanger confirming installation
 d) If user confirms the installation than the front-end will call DOMApplicationRegistry.confirmInstall(..) and the app will get installed.

Fabrice, please correct me if I'm wrong here


So this bug is primarily about (c), and possible follow-ups for handling exception and error cases, if any (I'll be looking at this soon).

A bunch of this was previously implemented in the patch at bug 697006, in the webappsUI.jsm file, and parts of it are reviewed!  I'm not convinced though at the approach of using a .jsm for UI code like this, but it was used to handle the global (not per-window) nature of the observer system

Comment 2

[:Felipe Gomes (needinfo for replies!)]

Attachment: Screenshot

Here's a screenshot of the doorhanger as currently implemented by the add-on. The add-on can be installed here: http://people.mozilla.com/~dclarke/openwebapps/Extension/ and the code is here: https://github.com/mozilla/openwebapps/tree/develop/addons/jetpack

I'm using this URL to test: https://stage-myapps.mozillalabs.com/apps/appdir/

Comment 3

[[:fabrice] Fabrice Desré]

This is a dupe of bug 697006 for which I have a rebased patch. It implements (c) in comment #1