1118138 - B2G desktop should be able to run non-zipped apps

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Runtime, defect)

Description

[Paul Rouget [:paul]]

We want to be able to point B2G to a manifest file to run an app as a package app. Today, an app needs to be a zip file (application.zip).

Then, working on an app would not require a build step.

Comment 1

[Alexandre Poirot [:ochameau]]

It would also help getting rid of the DEBUG=1 mode in gaia where we involve a local http server to work around this.

Comment 2

[Alexandre Poirot [:ochameau]]

Attachment: patch v1

Allow loading webapps from a folder.
But that doesn't work yet for unknown reason.
I used this gaia hack in order to set the magic flag in webapps.json:
  https://github.com/ochameau/gaia/commit/fcaff0b0070061ad9766a4fa80602e2ba8f0ddf8

Comment 3

[Alexandre Poirot [:ochameau]]

I see two request correctly resolved to:
  file:///home/alex/gaia/profile/webapps/system.gaiamobile.org/index.html
But then, nothing, no particular error and black screen.

Comment 4

[Paul Rouget [:paul]]

(In reply to Alexandre Poirot [:ochameau] from comment #3)
> I see two request correctly resolved to:
>   file:///home/alex/gaia/profile/webapps/system.gaiamobile.org/index.html
> But then, nothing, no particular error and black screen.

Random guess… don't you hit this:

http://mxr.mozilla.org/mozilla-central/source/netwerk/ipc/NeckoParent.cpp#636

Comment 5

[[:fabrice] Fabrice Desré]

Attachment: launch-manifest.patch

This patch adds support for a --start-manifest command line parameter. When it is present, we use that manifest url as the system app to bootstrap from.

I intentionally didn't make that support desktop specific, though we'll need to add resource preloading for proper device support.

Comment 6

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

Comment on attachment 8552043 [details] [diff] [review]
launch-manifest.patch

Review of attachment 8552043 [details] [diff] [review]:
-----------------------------------------------------------------

::: b2g/chrome/content/shell.js
@@ +224,5 @@
>  
> +  bootstrap: function() {
> +    let startManifestURL =
> +      Cc['@mozilla.org/commandlinehandler/general-startup;1?type=b2gbootstrap']
> +        .getService(Ci.nsISupports).wrappedJSObject.startManifestURL;

This is a pretty ugly line.

::: b2g/components/Bootstraper.jsm
@@ +15,5 @@
> +Cu.import("resource://gre/modules/Webapps.jsm");
> +Cu.import("resource://gre/modules/AppsUtils.jsm");
> +
> +function debug(aMsg) {
> +  dump("-*- Bootstraper: " + aMsg + "\n");

//

Comment 7

[[:fabrice] Fabrice Desré]

https://hg.mozilla.org/integration/b2g-inbound/rev/c636c1f0e07b

Comment 8

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

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