Closed
Bug 745924
Opened 13 years ago
Closed 12 years ago
Allow launching installed native apps from WebappsInstaller
Categories
(Firefox Graveyard :: Web Apps, enhancement, P1)
Firefox Graveyard
Web Apps
Tracking
(blocking-kilimanjaro:+)
RESOLVED
DUPLICATE
of bug 772600
blocking-kilimanjaro | + |
People
(Reporter: Mardak, Unassigned)
References
(Blocks 1 open bug)
Details
Bug 740922 removes the pinned tab behavior of launching apps, but the desired behavior is to launch the native app. After the native installers for windows bug 731541 and os x bug 739636, WebappsInstaller could expose a way to launch a manifest if it's natively installed.
Felipe has pointed out that there might be multiple installs of the same app (perhaps across users?) so each Firefox profile needs to keep track of which app (and associated receipts) to launch.
Dan has pointed out that OS X allows the user to move the apps anywhere so launching by bundle id is the correct approach, but I'm not sure how that ties in with Felipe's concern of multiple installations.
Reporter | ||
Comment 1•13 years ago
|
||
One way to launch from OS X is with Apple Script using Finder. But this doesn't deal with deleted app files or duplicate native app installs.
let process = Cc["@mozilla.org/process/util;1"].createInstance(Ci.nsIProcess);
let file = Cc["@mozilla.org/file/local;1"].createInstance(Ci.nsILocalFile); file.initWithPath("/usr/bin/osascript");
process.init(file);
process.runAsync(['-e', 'tell application "Finder" to open application file id "http://marblerun.at"'], 2);
Where http://marblerun.at is the CFBundleIdentifier from <string>' + escapeXML(this.launchURI.prePath) + '</string>
Comment 2•13 years ago
|
||
Flagging as enhancement - This does sound important, although not required for FF 14.
Severity: normal → enhancement
Comment 3•13 years ago
|
||
So by putting together the two requirements I think the correct thing to do is to, upon install, store the platform-specific "path" into the webapps.json file (through DOMApplicationRegistry).
On OSX for example, the "path" would be the bundle id that is to be used to launch the app.
Right now that means we don't support having more than 1 app with the same name, but the could will already be correct whenever the generated bundle ids support this
Updated•13 years ago
|
Whiteboard: [marketplace-beta-]
Comment 4•13 years ago
|
||
felipe, in OSX, app names are purely user-facing. App identification is done the with BundleID. So you could actually have many apps with the same name. If they also have the same BundleID, then which one is launched is up to the OS.
Comment 5•13 years ago
|
||
Nominating for k9o, as this is likely going to be needed for new tab and awesome bar integration for firefox integration with web apps. This feature would allow launching a native application within firefox in other words.
blocking-kilimanjaro: --- → ?
Updated•13 years ago
|
blocking-kilimanjaro: ? → +
Updated•13 years ago
|
Whiteboard: [marketplace-beta-]
Updated•13 years ago
|
Target Milestone: --- → Future
Updated•13 years ago
|
Updated•13 years ago
|
Priority: -- → P1
Updated•12 years ago
|
Target Milestone: Future → ---
Comment 6•12 years ago
|
||
Making this the meta bug - we'll build separate bugs for each implementation of this - OS X already exists as bug 763740.
Keywords: meta
Comment 7•12 years ago
|
||
Marking for security review, per discussions with rs and myk, and per [1].
Security team: This meta bug and its implementation bugs involve Firefox launching processes on the user's machine. It seems wise to review whether we're doing this securely.
[1] https://wiki.mozilla.org/Security/Reviews/Review_Request_Form#Security_Assurance_Security_Review_Request
Keywords: sec-review-needed
Updated•12 years ago
|
QA Contact: jsmith
Updated•12 years ago
|
Whiteboard: [sec-assigned:curtisk]
Updated•12 years ago
|
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → DUPLICATE
Comment 9•12 years ago
|
||
Moving the sec flags to bug 772600 (the dup of this bug).
Keywords: meta,
sec-review-needed
Whiteboard: [sec-assigned:curtisk]
Assignee | ||
Updated•9 years ago
|
Product: Firefox → Firefox Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•