Closed
Bug 757613
Opened 12 years ago
Closed 12 years ago
Use manifest.webapp instead of manifest.json
Categories
(Core Graveyard :: DOM: Apps, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla15
People
(Reporter: fabrice, Assigned: fabrice)
Details
Attachments
(1 file)
1.21 KB,
patch
|
anant
:
review+
|
Details | Diff | Splinter Review |
We promote the use of manifest.webapp everywhere (and switched to that in gaia), so let's align the implementation here also. This uses .webapp when installing new apps, and supports .json and .webapp when reading to not lose previously installed apps.
Attachment #626181 -
Flags: review?(anant)
Comment 1•12 years ago
|
||
Comment on attachment 626181 [details] [diff] [review] patch Looks good to me, though it does feel a bit clunky having that one bit of backwards-compatibility code in there. Hopefully we won't need it anymore once some form of app sync lands.
Attachment #626181 -
Flags: review?(anant) → review+
Assignee | ||
Comment 2•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/90f5320604af
Comment 3•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/90f5320604af
Assignee: nobody → fabrice
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla15
Updated•12 years ago
|
Whiteboard: [qa+]
Comment 4•12 years ago
|
||
This patch only modifies the DOM Registry, correct? If so, is there any way to verify this from an end-user perspective? I know installing an app on desktop still produces a "manifest.json" in the APPDATA folder for windows with this patch.
Whiteboard: [qa+] → [qa?]
Updated•12 years ago
|
Component: DOM: Mozilla Extensions → DOM: Apps
Updated•12 years ago
|
Whiteboard: [qa?]
Updated•7 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•