Closed Bug 104402 Opened 23 years ago Closed 18 years ago

eliminate nsIChromeEventHandler interface from XUL element

Categories

(Core :: XUL, defect, P3)

defect

Tracking

()

RESOLVED FIXED
Future

People

(Reporter: waterson, Assigned: smaug)

References

(Blocks 1 open bug)

Details

(Keywords: helpwanted, memory-footprint)

hyatt, was this the interface we wanted to nix from XUL element? Or was it
nsIChromeEventHandler?
Blocks: 104400
Status: NEW → ASSIGNED
Keywords: footprint
Chrome event handler.
Summary: eliminate nsIDOMEventReciever interface from XUL element → eliminate nsIChromeEventHandler interface from XUL element
Target Milestone: --- → mozilla0.9.7
So, ahh, hrm. I'm going to need a little guidance here. Could I simply eliminate
the nsIChromeEventHandler interface altogether, and use the nsIContent interface
(calling HandleDOMEvent directly)? Oops, no nsWindowRoot stuffs that idea.
Should we make a subclass of nsXULElement that implements this for <browser>'s?
(I suppose that's what this is for.) Anyway, vend me a clue.

Also, looking at this makes me sad that we added nsIScriptEventHandlerOwner. I
wonder if that functionality could've been rolled into another interface (like
nsIScriptObjectOwner). Finally, it's too bad that nsIDOMEventReceiver and
nsIDOMEventTarget have to be separate interfaces, since I bet all elements can
be targets. Maybe we did this because documents can be event targets, too (but
documents aren't elements, even though the look sort of like elements). Ah to be
back in the salad days when ifaces were loose and free.
Priority: -- → P3
Target Milestone: mozilla0.9.7 → mozilla0.9.8
bryner, here's a footprint win that would be splendid to get in. I have no idea
how hyatt intended to do this. If you're not interested, send it back to me and
I'll deal with it after sabbatical.
Assignee: waterson → bryner
Status: ASSIGNED → NEW
Target Milestone: mozilla0.9.8 → ---
I'm interested, but I talked this over with hyatt and this is definitely
non-trivial.  I'm loaded up with XBL form control stuff at the moment, so I'm
not going to have time to get to this anytime soon.  Back to you, but I'll ping
you about this if I end up having time to work on it.
Assignee: bryner → waterson
shaver?
Status: NEW → ASSIGNED
Keywords: helpwanted
Target Milestone: --- → Future
sicking, weren't we just talking about this?  ;)

Assignee: waterson → Olli.Pettay
Status: ASSIGNED → NEW
Depends on: 363067
Fixed by Bug 363067.
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Component: XP Toolkit/Widgets: XUL → XUL
QA Contact: jrgmorrison → xptoolkit.widgets
You need to log in before you can comment on or make changes to this bug.