Open Bug 517384 Opened 15 years ago Updated 2 years ago

Reevaluate jscontext pushing for event handling

Categories

(Core :: DOM: Events, defect, P5)

Product:
Core
Component:
DOM: Events
Platform:
x86
macOS
Type:
defect

Tracking

()

People

(Reporter: bzbarsky, Unassigned)

References

Details

(Keywords: perf)

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?
Ben, you added some cx pushes in bug 482306.
Bug 517879 should help here a bit.
Depends on: 517879
https://bugzilla.mozilla.org/show_bug.cgi?id=1472046 Move all DOM bugs that haven’t been updated in more than 3 years and has no one currently assigned to P5. If you have questions, please contact :mdaly.
Priority: -- → P5
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.