Move WebappsInstaller from browser/modules to toolkit/webapps

RESOLVED FIXED in Firefox 17

Status

Firefox Graveyard
Web Apps
P2
normal
RESOLVED FIXED
5 years ago
a year ago

People

(Reporter: Felipe, Assigned: Felipe)

Tracking

Trunk
Firefox 17

Details

(Whiteboard: [qa-])

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
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.
(Assignee)

Comment 1

5 years ago
Created attachment 644106 [details] [diff] [review]
Patch

Updated

5 years ago
status-firefox16: --- → wontfix

Updated

5 years ago
Priority: -- → P2
(Assignee)

Comment 2

5 years ago
Comment on attachment 644106 [details] [diff] [review]
Patch

See comment 0
Attachment #644106 - Flags: review?(mh+mozilla)
Attachment #644106 - Flags: review?(mh+mozilla) → review+
(Assignee)

Comment 3

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/01f85b0fe9fc
Target Milestone: --- → Firefox 17

Updated

5 years ago
Whiteboard: [qa-]
https://hg.mozilla.org/mozilla-central/rev/01f85b0fe9fc
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.