35707 - Live Connect should be lazily initialized

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect, P3)

Description

[travis]

Ok, so the plugin DLL is still coming in basically on the creation of the first 
JSEnvironment.  This is due to the fact that live connect is not being lazily 
initialized.  The line of code is 
http://lxr.mozilla.org/seamonkey/source/dom/src/base/nsJSEnvironment.cpp#1065.  
By Brendan's suggestion I'm giving it to Clayton.

Comment 1

[clayton]

Passing the buck to Norris.   = C =
 

Comment 2

[Norris Boyd]

Roger, could you or rginda own this?

Comment 3

[Mike McCabe]

Seems like we should be able to skip the init of liveconnect and register
'java', 'sun' and 'netscape' with the namespace manager, and have *them*
initialize it on demand.

Reassigning to jst, moving to layout.

Comment 4

[Phil Schwartau]

Changing QA contact to reflect reassignment -

Comment 5

[Brendan Eich [:brendan]]

McCabe's got the right idea.  Just don't forget to register "Packages" too.

/be

Comment 6

[Johnny Stenback (:jst)]

Accepting bug and nominating for beta3, fixing this would cut down on startup
time and we're badly in need of cutting down on that...

Comment 7

[Nisheeth Ranjan]

Do we have any data on how much time this will save on startup?  I'm inclined to 
mark this beta3-.  Any objections?

Comment 8

[Brendan Eich [:brendan]]

Argh!  Before you triage this prematurely, consider giving it to me.  I'll hack 
on it.

/be

Comment 9

[Nisheeth Ranjan]

Re-assigning to brendan (Johnny already has 19 nsbeta3+ bugs on his plate)...

Comment 10

[Brendan Eich [:brendan]]

Mine all mine.

/be

Comment 11

[Brendan Eich [:brendan]]

bug 46703 may be relevant: while that bug asks to share standard classes and
clone their constructors lazily from a superglobal, it also talks of lazily
creating (as well as binding) such objects.  This bug (35707) definitely wants
both late creation and late binding.

/be

Comment 12

[selmer (gone)]

Should this go back to jst now?  Any activity planned?  Nominating for mozilla0.9.

Comment 13

[Johnny Stenback (:jst)]

Taking this one back, I'll try to get to this for mozilla0.9

Comment 14

[Phil Schwartau]

Setting default QA contact -

Comment 15

[Johnny Stenback (:jst)]

Won't happen in time for mozilla0.9

Comment 16

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Moving bugs that do not have "important" keywords or fixes in hand to the next
milestone.

Comment 17

[Patrick C. Beard]

This is already done. We should no longer be seeing Java initialize at startup, 
until an applet loads, or explicit LiveConnect uses, such as 
java.lang.System.out.println('hello world.'). The DOM code we have right now is 
sufficiently lazy, because we need to have the LiveConnect properties be 
initialized.