Closed Bug 746771 Opened 11 years ago Closed 11 years ago

Mac installer needs to use updated webapp.json structure with registryDir instead of app.profile

Categories

(Firefox Graveyard :: Web Apps, defect)

Product:
Firefox Graveyard
Component:
Web Apps
Version:
14 Branch
Platform:
x86
macOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
VERIFIED FIXED
Milestone:
Firefox 14

People

(Reporter: Mardak, Assigned: Felipe)

References

Details

(Whiteboard: [qa!])

Attachments

(1 file, 1 obsolete file)

for checkin
1.12 KB, patch
Details | Diff | Splinter Review 
https://hg.mozilla.org/mozilla-central/rev/762911344837#l3.205
The code from bug 739636 creates a json object of format:
{
  app: {
    profile: ..,
    origin: ..,
    installOrigin: ..,
    manifest: ..
  }
}

The runtime from bug 725408 is expecting a top level registryDir property on this config file.

http://mxr.mozilla.org/mozilla-central/source/webapprt/WebappRTDirectoryProvider.js#24
Component: Desktop Runtime → Web Apps
Product: Web Apps → Firefox
QA Contact: desktop-runtime → webapps
Version: unspecified → 14 Branch
Dan - Any ideas?
Assignee: nobody → felipc
Attached patch Patch (obsolete) — — Splinter Review 
I'll wait for Mardak to confirm this is the correct fix before requesting review
Blocks: 746213
Attachment #616320 - Flags: review?(myk)
Comment on attachment 616320 [details] [diff] [review]
Patch

This is the right fix, per the install flow <https://etherpad.mozilla.org/webapprt-install-flow>.  However, also remove the `profile` property from the `app` structure, as it is redundant and unnecessary.
Attachment #616320 - Flags: review?(myk) → review+
Blocks: 731054
Attached patch for checkin — — Splinter Review
Attachment #616320 - Attachment is obsolete: true
Summary: Mac installer creates incorrect webapp.json → Mac installer needs to use updated webapp.json structure with registryDir instead of app.profile
https://hg.mozilla.org/mozilla-central/rev/ffb9beb36e5a
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 14
Whiteboard: [qa+]
No longer blocks: 731054
Flags: in-moztrap?(jsmith)
To verify this bug, I need to install an application and check that the webapp.json file contains registryDir, right?

For some reason when I do "Show Package Contents" on Mac, I can't find the webapp.json file. Is this expected?
webapp.json actually appears in the user's profile and not part of the application:

~/Library/Application Support/marketplace-dev.allizom.org;https;-1/webapp.json
No longer blocks: 746213
No longer depends on: 725408, 739636
Blocks: 746213
Depends on: 725408, 739636
Verified on OS X 10.7.
Status: RESOLVED → VERIFIED
Whiteboard: [qa+] → [qa!]
Flags: in-moztrap?(jsmith) → in-moztrap-
QA Contact: jsmith
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.