Closed Bug 678708 Opened 8 years ago Closed 8 years ago

nsMathMLmactionFrame::MouseListener::HandleEvent may abort in debug mode

Categories

(Core :: MathML, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla9

People

(Reporter: fredw, Assigned: atulagrwl)

References

(Blocks 1 open bug, )

Details

(Whiteboard: [good first bug])

Attachments

(1 file)

I remember to have been able to reach the NS_ABORT in debug mode. That does not seem to be what you want. Shouldn't we rather pass the event to another handler?
The NS_ABORT was added in
http://hg.mozilla.org/mozilla-central/rev/a41df1f97f0d

Listeners are added for "mouseover", "click", and "mouseout".  The code seems to expect that "click" changes to "mouseclick", but that might be a mistake.
Yes, that's just a typo, the "mouseclick" should be changed to "click"
Keywords: helpwanted
Whiteboard: [good first bug]
Attached patch Patch v1Splinter Review
Attachment #556751 - Flags: review?(karlt)
Attachment #556751 - Flags: review?(karlt) → review+
Assignee: nobody → atulagrwl
Status: NEW → ASSIGNED
In my queue :-)
Keywords: checkin-needed
http://hg.mozilla.org/mozilla-central/rev/256a8799fce7
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Flags: in-testsuite?
> Ed Morley [:edmorley] 2011-09-01 01:33:58 PDT 
> Flags: in-testsuite?

I wonder whether we can write a test page that uses Javascript to simulate the "click" event on a maction element.

https://developer.mozilla.org/en/DOM/element.dispatchEvent
(Clearing to stop this request showing up on the 'My Requests' page)
Flags: in-testsuite?
You need to log in before you can comment on or make changes to this bug.