706406 - <maction> doesn't work when click event is called stopPropagation()

Status: RESOLVED FIXED

(Core :: MathML, defect)

Description

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Load the URI, you cannot toggle the maction's content.

Comment 1

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Attachment: Patch

This also fixes another bug the frame doesn't check defaultPrevented value.

Comment 2

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

(In reply to Masayuki Nakano (:masayuki) (Mozilla Japan) from comment #1)
> Created attachment 577930 [details] [diff] [review] [diff] [details] [review]
> Patch
> 
> This also fixes another bug the frame doesn't check defaultPrevented value.

Does this fix bug 433774?

Comment 3

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

(In reply to Frédéric Wang from comment #2)
> (In reply to Masayuki Nakano (:masayuki) (Mozilla Japan) from comment #1)
> > Created attachment 577930 [details] [diff] [review] [diff] [details] [review] [diff] [details] [review]
> > Patch
> > 
> > This also fixes another bug the frame doesn't check defaultPrevented value.
> 
> Does this fix bug 433774?

On the testcase, no. The click event is fired on <mn> rather than <maction>. So, the testcase doesn't make sense.

Comment 4

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

But that is a good point, I'll land the patch without defaultPrevented code, and I'll attach a new patch to the bug.

Comment 5

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/a8192280c3b3

Comment 6

[Matt Brubeck (:mbrubeck)]

https://hg.mozilla.org/mozilla-central/rev/a8192280c3b3