Last Comment Bug 678708 - nsMathMLmactionFrame::MouseListener::HandleEvent may abort in debug mode
: nsMathMLmactionFrame::MouseListener::HandleEvent may abort in debug mode
Status: RESOLVED FIXED
[good first bug]
:
Product: Core
Classification: Components
Component: MathML (show other bugs)
: Trunk
: All All
: -- normal (vote)
: mozilla9
Assigned To: Atul Aggarwal
:
: Anthony Jones (:kentuckyfriedtakahe, :k17e)
Mentors:
http://hg.mozilla.org/mozilla-central...
Depends on:
Blocks: maction 664058
  Show dependency treegraph
 
Reported: 2011-08-13 04:36 PDT by Frédéric Wang (:fredw)
Modified: 2012-09-14 03:07 PDT (History)
3 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
Patch v1 (746 bytes, patch)
2011-08-29 21:48 PDT, Atul Aggarwal
karlt: review+
Details | Diff | Splinter Review

Description Frédéric Wang (:fredw) 2011-08-13 04:36:04 PDT
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 (back Dec 13 :karlt) 2011-08-14 19:29:25 PDT
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 2011-08-14 21:17:47 PDT
Yes, that's just a typo, the "mouseclick" should be changed to "click"
Comment 3 Atul Aggarwal 2011-08-29 21:48:31 PDT
Created attachment 556751 [details] [diff] [review]
Patch v1
Comment 4 Ed Morley [:emorley] 2011-08-31 05:09:27 PDT
In my queue :-)
Comment 6 Ed Morley [:emorley] 2011-09-01 01:33:08 PDT
http://hg.mozilla.org/mozilla-central/rev/256a8799fce7
Comment 7 Frédéric Wang (:fredw) 2011-09-01 08:46:39 PDT
> 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] 2012-09-14 03:07:20 PDT
(Clearing to stop this request showing up on the 'My Requests' page)

Note You need to log in before you can comment on or make changes to this bug.