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

RESOLVED WORKSFORME

Status

()

Core
DOM: Core & HTML
P3
normal
RESOLVED WORKSFORME
19 years ago
7 years ago

People

(Reporter: Dan M, Unassigned)

Tracking

({dom0})

Trunk
Future
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

19 years ago
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.
(Reporter)

Updated

19 years ago
Status: NEW → ASSIGNED
Target Milestone: M9
(Reporter)

Updated

19 years ago
Target Milestone: M9 → M10

Updated

19 years ago
Component: Browser-General → Javascript Engine
QA Contact: leger → cbegle

Updated

19 years ago
QA Contact: cbegle → desale
(Reporter)

Updated

19 years ago
Target Milestone: M10 → M12

Comment 1

19 years ago
mass-moving all m12 bugs to m13
(Reporter)

Updated

19 years ago
Target Milestone: M13 → M15
(Reporter)

Updated

19 years ago
Target Milestone: M15 → M17

Comment 2

19 years ago
Mass moving M17 bugs to M18
Target Milestone: M17 → M18

Comment 3

18 years ago
mass-moving all bugs to m21 that are not dofood+, or nsbeta2+
Target Milestone: M18 → M21
(Reporter)

Updated

18 years ago
Target Milestone: M21 → Future

Comment 4

18 years ago
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

Comment 6

18 years ago
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
(Reporter)

Updated

14 years ago
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
Last Resolved: 7 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.