Closed Bug 775803 Opened 8 years ago Closed 7 years ago

Move WebappsInstaller from browser/modules to toolkit/webapps

Categories

(Firefox Graveyard :: Web Apps, defect, P2)

defect

Tracking

(firefox16 wontfix)

RESOLVED FIXED
Firefox 17
Tracking Status
firefox16 --- wontfix

People

(Reporter: Felipe, Assigned: Felipe)

Details

(Whiteboard: [qa-])

Attachments

(1 file)

The WebappsInstaller code is mostly browser code. However, it's used by both firefox and webapprt. With the imminent move of app resources in bug 755724, it's my understading that the webapprt won't be able to access anymore the files packaged in browser. Benjamin or Glandium, is that correct?

If so, we need to move it toolkit to keep app installation from inside webapprt working. Moving it there has other advantages as well, for example Wesj wanted access to some of the functions in WebappsIconHelper that this patch moves as well.
Attached patch PatchSplinter Review
Priority: -- → P2
Comment on attachment 644106 [details] [diff] [review]
Patch

See comment 0
Attachment #644106 - Flags: review?(mh+mozilla)
Attachment #644106 - Flags: review?(mh+mozilla) → review+
Whiteboard: [qa-]
https://hg.mozilla.org/mozilla-central/rev/01f85b0fe9fc
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.