Closed
Bug 1248734
Opened 9 years ago
Closed 9 years ago
Fix SDK startup on Instanbird
Categories
(Add-on SDK Graveyard :: General, defect)
Add-on SDK Graveyard
General
Tracking
(firefox48 fixed)
RESOLVED
FIXED
mozilla48
Tracking | Status | |
---|---|---|
firefox48 | --- | fixed |
People
(Reporter: freaktechnik, Unassigned)
Details
Attachments
(1 file)
1.79 KB,
patch
|
zer0
:
review+
|
Details | Diff | Splinter Review |
Add-on SDK extensions currently fail to load on startup on Instantbird (a comm-central application). This patch adds Instantbird as a detectable XUL app and gives it a custom observer topic to listen to on startup. Adding it to the detectable XUL apps is needed to have a custom observer topic.
Reporter | ||
Updated•9 years ago
|
Attachment #8719938 -
Flags: review?(zer0)
Comment 1•9 years ago
|
||
Comment on attachment 8719938 [details] [diff] [review]
sdk-instantbird-startup.patch
Review of attachment 8719938 [details] [diff] [review]:
-----------------------------------------------------------------
Sorry for the long waiting; Looks good to me. Maybe we should point what is "officially" supported, but I think at this point is not really necessary.
Attachment #8719938 -
Flags: review?(zer0) → review+
Updated•9 years ago
|
Keywords: checkin-needed
Reporter | ||
Comment 2•9 years ago
|
||
(In reply to Matteo Ferretti [:matteo][:zer0] from comment #1)
> Sorry for the long waiting; Looks good to me. Maybe we should point what is
> "officially" supported, but I think at this point is not really necessary.
Thabk you for the review! I think there's an MDN page somewhere touching on the subject of SDK extensions outside of Firefox. I'll look it up and extend it if it makes sense.
Keywords: checkin-needed
Comment 4•9 years ago
|
||
bugherder |
Status: UNCONFIRMED → RESOLVED
Closed: 9 years ago
status-firefox48:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
You need to log in
before you can comment on or make changes to this bug.
Description
•