Reevaluate jscontext pushing for event handling

NEW
Unassigned

Status

()

Core
DOM: Events
9 years ago
9 years ago

People

(Reporter: bz, Unassigned)

Tracking

(Blocks: 1 bug, {perf})

Trunk
x86
Mac OS X
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Do we need to push a JSContext in nsEventListenerManager::CompileEventHandlerInternal?  We push one anyway in BindCompiledEventHandler; do we need the jscontext pushed when calling CompileEventHandler?  Can we remove the push in BindCompiledEventHandler if it's redundant?

For that matter, do we need the push in CallEventHandler?

Comment 1

9 years ago
Ben, you added some cx pushes in bug 482306.

Comment 2

9 years ago
Bug 517879 should help here a bit.
Depends on: 517879
You need to log in before you can comment on or make changes to this bug.