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

Status: RESOLVED FIXED

(Core :: MathML, defect)

Description

[Frédéric Wang (:fredw)]

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?

Comment 1

[Karl Tomlinson (:karlt)]

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.

Comment 2

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Yes, that's just a typo, the "mouseclick" should be changed to "click"

Comment 3

[Atul Aggarwal]

Attachment: Patch v1

Comment 4

[Ed Morley [:emorley]]

In my queue :-)

Comment 5

[Ed Morley [:emorley]]

http://hg.mozilla.org/integration/mozilla-inbound/rev/256a8799fce7

Comment 6

[Ed Morley [:emorley]]

http://hg.mozilla.org/mozilla-central/rev/256a8799fce7

Comment 7

[Frédéric Wang (:fredw)]

> 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

Comment 8

[Ed Morley [:emorley]]

(Clearing to stop this request showing up on the 'My Requests' page)