Closed
Bug 877413
Opened 11 years ago
Closed 11 years ago
Add an addPath to toolkit/loader
Categories
(Add-on SDK Graveyard :: General, defect, P1)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: dcamp, Unassigned)
Details
Attachments
(1 file)
We want the devtools server loader to load all extensions to the server in its own loader, for a few reasons:
a) We want to be able to identify all globals associated with the loader so we can avoid debugging ourselves during chrome debugging.
b) We want to be able to load multiple copies of the server so that we can debug the debugging server.
So we want tests and extensions to be able to add URIs that can be loaded. A few options that would be acceptable are:
a) require("resource://fully/qualified/url");
b) defineModule(loader, "extensionpath/module", "resource://extensionuri/module.js"); require("extensionpath/module");
c) addPath(loader, "extensionpath/", "resource://extensionuri/"); require("extensionpath/module");
c) seems the simplest, so I'll send a pull request that does that.
Reporter | ||
Comment 1•11 years ago
|
||
Pointer to Github pull-request
Priority: -- → P1
Want to assign this to Irakli for a review, Dave?
Reporter | ||
Comment 3•11 years ago
|
||
Comment on attachment 755623 [details]
Pointer to Github pull request: https://github.com/mozilla/addon-sdk/pull/1012#attch-to-bugzilla
Yeah, got distracted.
Attachment #755623 -
Flags: review?(rFobic)
Comment 4•11 years ago
|
||
Comment on attachment 755623 [details]
Pointer to Github pull request: https://github.com/mozilla/addon-sdk/pull/1012#attch-to-bugzilla
Removing review request for now since after chatting on IRC dcamp decide to try using `resolve` + `mapping` instead.
Dave feel free to re-request if that'll becomes more complicated than necessary.
Attachment #755623 -
Flags: review?(rFobic)
Reporter | ||
Comment 6•11 years ago
|
||
I'm not sure we still need this - we can reopen if we do.
Status: NEW → RESOLVED
Closed: 11 years ago
Flags: needinfo?(dcamp)
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•