Closed
Bug 779935
Opened 12 years ago
Closed 12 years ago
nsIAppsService.idl need to get mozIDOMApplication from manifest id
Categories
(Core :: DOM: Device Interfaces, defect)
Core
DOM: Device Interfaces
Tracking
()
RESOLVED
FIXED
mozilla17
People
(Reporter: dougt, Assigned: fabrice)
References
Details
Attachments
(1 file, 1 obsolete file)
|
(deleted),
patch
|
mounir
:
review+
|
Details | Diff | Splinter Review |
We need the ablity to go from a manifest id (which is part of the nsIPrincipal) to a mozIDOMApplication. The current methods on nsIAppsService only accept manifest urls.
|
Assignee | |
Comment 1•12 years ago
|
||
How are these ids created?
|
||
Updated•12 years ago
|
OS: Mac OS X → All
Hardware: x86 → All
|
|
||
Comment 2•12 years ago
|
||
(In reply to Fabrice Desré [:fabrice] from comment #1)
> How are these ids created?
This is |localId| I believe.
|
|
Assignee | |
Comment 3•12 years ago
|
||
|
|
Assignee | |
Updated•12 years ago
|
Attachment #649330 -
Flags: review?(mounir)
|
||
Updated•12 years ago
|
|
|
||
Comment 4•12 years ago
|
||
Comment on attachment 649330 [details] [diff] [review]
patch
Review of attachment 649330 [details] [diff] [review]:
-----------------------------------------------------------------
Could you add a method that also returns the manifest URL from the localId? I believe some consumers only need that.
Attachment #649330 -
Flags: review?(mounir) → review+
|
|
Assignee | |
Comment 5•12 years ago
|
||
Updated to add a |getManifestURLByLocalId(id)| method.
Attachment #649330 -
Attachment is obsolete: true
Attachment #649854 -
Flags: review?(mounir)
|
|
||
Comment 6•12 years ago
|
||
Comment on attachment 649854 [details] [diff] [review]
patch v2
Review of attachment 649854 [details] [diff] [review]:
-----------------------------------------------------------------
Can you push this ASAP so dependencies don't get blocked for too long.
Attachment #649854 -
Flags: review?(mounir) → review+
|
|
Assignee | |
Comment 7•12 years ago
|
||
|
||
Comment 8•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
You need to log in
before you can comment on or make changes to this bug.
Description
•