Closed Bug 10029 Opened 26 years ago Closed 13 years ago

encapsulate JS incantations in the C++ window.open code

Categories

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

defect

Tracking

()

RESOLVED WORKSFORME
Future

People

(Reporter: danm.moz, Unassigned)

Details

(Keywords: dom0)

JS window.open can be called from C++, but only by adding a bit of unpleasant-looking boilerplate
for doing the arguments conversion.  This is also causing undesired linkage to the JS library in certain
modules.  The boilerplate wants to be encapsulated in a single, easier to use C++ function.
Status: NEW → ASSIGNED
Target Milestone: M9
Target Milestone: M9 → M10
Component: Browser-General → Javascript Engine
QA Contact: leger → cbegle
QA Contact: cbegle → desale
Target Milestone: M10 → M12
mass-moving all m12 bugs to m13
Target Milestone: M13 → M15
Target Milestone: M15 → M17
Mass moving M17 bugs to M18
Target Milestone: M17 → M18
mass-moving all bugs to m21 that are not dofood+, or nsbeta2+
Target Milestone: M18 → M21
Target Milestone: M21 → Future
Not JS Engine; updating component to DOM Level 0
Component: Javascript Engine → DOM Level 0
Isn't alecf fixing this under the aegis of another bug?

/be
yep.. we never did figure out if we were going to do this in nsIScriptContext
(it was a generic method to call arbitrary context-oriented functions

I'll attach a patch which adds this to nsIScriptContext, but I'm wondering if
this should go either in it's own interface, or in some sort of
nsIJSScriptContext interface.
Keywords: dom0
Assignee: danm.moz → nobody
Status: ASSIGNED → NEW
Filter on "Nobody_NScomTLD_20080620"
QA Contact: desale → general
nsIDOMWindow.open is available for C++ callers.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.