Closed
Bug 772162
Opened 12 years ago
Closed 12 years ago
Disable webapps support on FF 15
Categories
(Firefox for Android Graveyard :: Web Apps (PWAs), defect, P1)
Tracking
(firefox15+ verified)
VERIFIED
FIXED
People
(Reporter: jsmith, Assigned: wesj)
References
Details
(Whiteboard: [blocking-webrtandroid1+])
Attachments
(1 file)
828 bytes,
patch
|
blassey
:
review+
lsblakk
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
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.
Reporter | ||
Updated•12 years ago
|
tracking-fennec: --- → ?
Reporter | ||
Comment 1•12 years ago
|
||
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.
Reporter | ||
Updated•12 years ago
|
tracking-fennec: ? → ---
tracking-firefox15:
--- → ?
Updated•12 years ago
|
Comment 2•12 years ago
|
||
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.
Assignee: nobody → blassey.bugs
Comment 3•12 years ago
|
||
Wes can track this. Take a look at what desktop did to disable. We probably want to mimic that.
Assignee: blassey.bugs → wjohnston
Reporter | ||
Updated•12 years ago
|
Priority: -- → P1
Whiteboard: [blocking-webrtandroid1+]
Reporter | ||
Updated•12 years ago
|
Blocks: Blocking-FFA-WebRT1+
Assignee | ||
Comment 5•12 years ago
|
||
Webapps.manifest register the navigator properties, so I think (based on the desktop patch) this is all we need.
Attachment #645933 -
Flags: review?(blassey.bugs)
Updated•12 years ago
|
Attachment #645933 -
Flags: review?(blassey.bugs) → review+
Assignee | ||
Comment 6•12 years ago
|
||
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.
Attachment #645933 -
Flags: approval-mozilla-beta?
Comment 7•12 years ago
|
||
Comment on attachment 645933 [details] [diff] [review] Patch Please update the status for firefox15 to fixed when this lands.
Attachment #645933 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Assignee | ||
Updated•12 years ago
|
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•12 years ago
|
status-firefox14:
--- → fixed
Assignee | ||
Updated•12 years ago
|
status-firefox14:
fixed → ---
status-firefox15:
--- → fixed
Reporter | ||
Updated•12 years ago
|
Whiteboard: [blocking-webrtandroid1+] → [blocking-webrtandroid1+], [qa+]
Assignee | ||
Comment 9•12 years ago
|
||
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.
Updated•12 years ago
|
Status: RESOLVED → VERIFIED
Updated•12 years ago
|
Whiteboard: [blocking-webrtandroid1+], [qa+] → [blocking-webrtandroid1+]
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•