The webapp runtime should read the webapp.json file at startup and then close it

RESOLVED FIXED in Firefox 27

Status

RESOLVED FIXED
5 years ago
2 years ago

People

(Reporter: marco, Assigned: marco)

Tracking

Trunk
Firefox 27
All
Windows 7

Details

Attachments

(1 attachment, 1 obsolete attachment)

The webapp runtime should read the webapp.json file at startup and then close it, so that we can update it later.
Created attachment 810675 [details] [diff] [review]
Patch

This should work, I'll test asap.
I guess it'd be better to move this IO off the main thread, we can do it in another bug (it's not a straightforward fix because the getter is synchronous, we need to modify the callers).
Assignee: nobody → mcastelluccio
Status: NEW → ASSIGNED
Attachment #810675 - Flags: review?(myk)
Comment on attachment 810675 [details] [diff] [review]
Patch

Nice catch!
Attachment #810675 - Flags: review?(myk) → review+
Created attachment 812307 [details] [diff] [review]
Patch

Updated commit message.
Attachment #810675 - Attachment is obsolete: true
Attachment #812307 - Flags: review+
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/73aee2b3959f
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 27
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.