772162 - Disable webapps support on FF 15

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: Web Apps (PWAs), defect, P1)

Description

[Jason Smith [:jsmith]]

Webapps support for FF for Android will not be ready in time for FF 15, so let's disable the functionality for it. We need to disable the install prompt and the management of apps.

Comment 1

[Jason Smith [:jsmith]]

Note for tracking nom - Android WebRT triage decided that we will not support web apps for FF for Android in FF 15, but we do not want this functionality active.

Comment 2

[Lukas Blakk [:lsblakk] use ?needinfo]

Assigning to blassey for now as the point person for disabling when the time comes. Brad, if there's someone else who could/should be on this feel free to re-assign to them.

Comment 3

[Mark Finkle (:mfinkle) (use needinfo?)]

Wes can track this. Take a look at what desktop did to disable. We probably want to mimic that.

Comment 4

[Wesley Johnston (:wesj)]

Desktop is bug 772638.

Comment 5

[Wesley Johnston (:wesj)]

Attachment: Patch

Webapps.manifest register the navigator properties, so I think (based on the desktop patch) this is all we need.

Comment 6

[Wesley Johnston (:wesj)]

Comment on attachment 645933 [details] [diff] [review]
Patch

[Approval Request Comment]
Bug caused by (feature/regressing bug #): None
User impact if declined: navigator.webapps methods will exist. could break sites that think that implies webapps support.
Testing completed (on m-c, etc.): Not landing on mc
Risk to taking this patch (and alternatives if risky): Low risk. same thing is on desktop firefox for 15.
String or UUID changes made by this patch: none.

Comment 7

[Lukas Blakk [:lsblakk] use ?needinfo]

Comment on attachment 645933 [details] [diff] [review]
Patch

Please update the status for firefox15 to fixed when this lands.

Comment 8

[Wesley Johnston (:wesj)]

https://hg.mozilla.org/releases/mozilla-beta/rev/aae7ded362d4

Comment 9

[Wesley Johnston (:wesj)]

Note this just kills navigator.apps stuff. I didn't remove about:apps. If that's wanted please say something.

There is no UI to get to it, and it will probably just fail to work without some of these navigator methods.