Closed Bug 902910 Opened 8 years ago Closed 8 years ago

Offer a way for simulator addon to integrate itself with the connection footer

Categories

(DevTools Graveyard :: WebIDE, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 26

People

(Reporter: ochameau, Assigned: ochameau)

Details

Attachments

(3 files, 2 obsolete files)

We need to define an API for the simulator addons to plug itself into the connection footer. We can have multiple simulator addons, mainly one per Firefox OS main releases like 1.1 and 1.2. We may also have other variations like having a partner specific builds,...

The connection footer should be able to:
 * help developer find'n install simulator addons,
 * discover installed simulator addons,
 * list them,
 * be able to invoke simulator code to launch the simulator and connect to it.
This work is also based on following simulator patch.
Attachment #792850 - Flags: review?(paul)
Attachment #792852 - Flags: review?(paul)
Attachment #792850 - Flags: review?(paul) → review+
Comment on attachment 792852 [details] [diff] [review]
Uplift simulator debugger client simulator fixes to be able to connect to it

Please re-base your code on the latest patch in bug 898485
Attachment #792852 - Flags: review?(paul) → review+
Rebased and also fixed a new exception in ConnectionManager._onTimeout (str is not defined)
Attachment #792852 - Attachment is obsolete: true
Comment on attachment 797138 [details] [diff] [review]
Uplift simulator debugger client simulator fixes to be able to connect to it

I hope you won't mind keeping the r+ with the rebase and just the naive onTimeout exception fix.
The other patch doesn't seem to require any rebase.
Attachment #797138 - Flags: review+
Whiteboard: [land-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/344094dcf795
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 26
So I forgot to land the first patch.
Product: Firefox → DevTools
Product: DevTools → DevTools Graveyard
You need to log in before you can comment on or make changes to this bug.