736312 - navigator.mozApps support on b2g

Status: RESOLVED FIXED

(Firefox OS Graveyard :: General, defect)

Description

[[:fabrice] Fabrice Desré]

Attachment: patch

Now that the new API has landed on m-c, we can add the glue to support it on b2g.

This patch enables the API, and maanges the launch() call by dispatching a "webapps-launch: event to the homescreen.

Comment 1

[[:fabrice] Fabrice Desré]

Attachment: test html file

Comment 2

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Comment on attachment 606402 [details] [diff] [review]
patch

>diff --git a/b2g/chrome/content/shell.js b/b2g/chrome/content/shell.js

>+
>+WebappsHelper = {
>+  init: function webapps_init() {
>+    Services.obs.addObserver(this, "webapps-launch", false);

The rest of the file use ' instead of "

>+  },
>+
>+  observe: function webapps_observe(aSubject, aTopic, aData) {

the rest of the file do not append the 'a' prefix

>+    let data = JSON.parse(aData);
>+    DOMApplicationRegistry.getManifestFor(data.origin, (function(aManifest) {
>+      if (!aManifest)
>+        return;

nit: add a line break

>+      let manifest = new DOMApplicationManifest(aManifest, data.origin);
>+      shell.sendEvent(content, "webapps-launch", { url: manifest.fullLaunchPath(), origin: data.origin } );

This is the stuff I'm not comfortable with. Is "webapps-launch" event describe somewhere on a spec? I would like to avoid to introduce 'proprietary' events.

>+    }).bind(this));

I don't think you need bind.

Comment 3

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

We don't have a spec defining how content is supposed to handle launching web apps.  We get to define it.

Not too thrilled with the approach here but it works for a v0.

Comment 4

[[:fabrice] Fabrice Desré]

Attachment: patch v2

updated patch according to IRC discussion

Comment 5

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Comment on attachment 606636 [details] [diff] [review]
patch v2

r+ if you fix the comments/nits I have done in the previous comment.

Comment 6

[[:fabrice] Fabrice Desré]

https://hg.mozilla.org/integration/mozilla-inbound/rev/e3efb4df551f

Comment 7

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/e3efb4df551f