648124 - Cache bootstrap.js scripts

Status: RESOLVED DUPLICATE of bug 648125

(Toolkit :: Add-ons Manager, defect)

Description

[Dave Townsend [:mossop]]

Currently we use loadSubScript which doesn't cache the parsed JS like XUL JS is. We could save startup time by doing this.

Comment 1

[Kris Maglione [:kmag]]

I'd also like to see these loaded into BackstagePass globals rather than Sandboxes, along with a requisite way to create said globals from JS. Sandboxes come with too much baggage, including a needless array of proxies for objects which pass in or out of them (and break things like E4X).

Comment 2

[Dave Townsend [:mossop]]

(In reply to comment #1)
> I'd also like to see these loaded into BackstagePass globals rather than
> Sandboxes, along with a requisite way to create said globals from JS.
> Sandboxes come with too much baggage, including a needless array of proxies
> for objects which pass in or out of them (and break things like E4X).

Should file a new bug about that, I wasn't really aware of any of those issues.

Comment 3

[Henrik Skupin [:whimboo][⌚️UTC+2]]

(In reply to comment #2)
> Should file a new bug about that, I wasn't really aware of any of those
> issues.

Dave, do you mean Kris or yourself?

Comment 4

[Dave Townsend [:mossop]]

(In reply to comment #3)
> (In reply to comment #2)
> > Should file a new bug about that, I wasn't really aware of any of those
> > issues.
> 
> Dave, do you mean Kris or yourself?

Kris should

Comment 5

[Kris Maglione [:kmag]]

Created bug 665810

Comment 6

[Michael Wu [:mwu]]

Bug 648125 caches all scripts loaded by loadSubScript.