Make COM connect a runtime option

VERIFIED WONTFIX

Status

()

VERIFIED WONTFIX
16 years ago
16 years ago

People

(Reporter: adamlock, Assigned: dbradley)

Tracking

Trunk
mozilla1.3beta
x86
Windows XP
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

16 years ago
COM connect is conditionally compiled with a build flag. Make COM connect build
by default but control its enabled/disabled state using a pref read at startup.
(Assignee)

Comment 1

16 years ago
So, we need to create a JS Component to serve as a service that implements an
interface defined by XPConnect. XPConnect creates the service, if that fails,
then it's a no, other wise it asks the service. The JS would then ask the prefs?

We need to ask this everytime we instantiate a COM object. So this isn't just at
startup.

Does that seem right Adam?
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla1.3beta
(Reporter)

Comment 2

16 years ago
It may be possible to extend the nsAxSecurityPolicy.js object with the attribute
you require. 

Is xpconnect able to read the prefs for itself or would this cause a dependency
issue?
(Assignee)

Comment 3

16 years ago
That would create another dependency problem. nsAxSecurityPolicy.js sounds like
a decent place to hang this, though.
(Assignee)

Comment 4

16 years ago
I was thinking about this again, and I think I need to have a seperate JS file.
This is because we may need to detect this when ActiveX is not present. For the
ActiveX it would work, but I'd like to put something in place that could be
usable for future projects that may not have ActiveX enabled.
What's the code footprint hit?

/be
(Assignee)

Comment 6

16 years ago
I'm not sure we'll switch the compile time option to be defaulted to on. I think
we only need to provide this for embedders that need it. So the runtime option
is in addition to the build time option. The embedders interested would have to
make the decision on whether they want to take the additional code to gain
IDispatch functionality.

I'll try and get some numbers together and post them to this bug.
(Assignee)

Comment 7

16 years ago
I don't believe that this should be built in regular Mozilla builds. I think
it's an optional feature that embedders can choose to support or not. The
ability to turn this on/off at runtime will be addressed with the patch for bug
188229
Status: ASSIGNED → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → WONTFIX

Comment 8

16 years ago
Marking Verified -
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.