Closed Bug 648124 Opened 10 years ago Closed 10 years ago

Cache bootstrap.js scripts

Categories

(Toolkit :: Add-ons Manager, defect)

defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 648125

People

(Reporter: mossop, Unassigned)

Details

(Whiteboard: [ts])

Currently we use loadSubScript which doesn't cache the parsed JS like XUL JS is. We could save startup time by doing this.
Whiteboard: [ts]
Depends on: 648125
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).
(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.
(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?
(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
Bug 648125 caches all scripts loaded by loadSubScript.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 648125
No longer depends on: 648125
You need to log in before you can comment on or make changes to this bug.