nsMathMLmactionFrame::Init reresolves its style context after munging an attribute... it's possible to craft style rules so that it'll actually end up with "display: none" after that, but still have a frame. That's bad.

I suggest to remove the actiontype="restyle". See bug 544036 comment 1

Agreed.

Comment on attachment 555085 [details] [diff] [review] remove nonstandard actiontype "restyle" of maction on MathML demo pages. >-×××¥ ×¢× ××¢××ר ××× ×ק×× ×××××ר ×צ××× ××× ××××××/××ק××× ××¢×רת RESTYLE: >+×××¥ ×¢× ××¢××ר ××× ×ק×× ×××××ר ×צ××× ××× ××××××/××ק×××: >-à¸à¸¥à¸´à¸à¸à¸£à¸à¹à¸«à¸à¸à¹à¹à¸à¹à¹à¸à¸à¸·à¹à¸à¸à¸µà¹à¸ªà¸µà¹à¸«à¸¥à¸·à¸­à¸ à¹à¸à¸·à¹à¸­ ยà¹à¸­/à¸à¸¢à¸²à¸¢ à¹à¸à¸¢à¹à¸à¹ RESTYLE: >+à¸à¸¥à¸´à¸à¸à¸£à¸à¹à¸«à¸à¸à¹à¹à¸à¹à¹à¸à¸à¸·à¹à¸à¸à¸µà¹à¸ªà¸µà¹à¸«à¸¥à¸·à¸­à¸ à¹à¸à¸·à¹à¸­ ยà¹à¸­/à¸à¸¢à¸²à¸¢ : Did you check that it is appropriate to remove these words in these languages?

(In reply to Karl Tomlinson (:karlt) from comment #5) > Comment on attachment 555085 [details] [diff] [review] > remove nonstandard actiontype "restyle" of maction on MathML demo pages. > > >-×××¥ ×¢× ××¢××ר ××× ×ק×× ×××××ר ×צ××× ××× ××××××/××ק××× ××¢×רת RESTYLE: > >+×××¥ ×¢× ××¢××ר ××× ×ק×× ×××××ר ×צ××× ××× ××××××/××ק×××: > > >-à¸à¸¥à¸´à¸à¸à¸£à¸à¹à¸«à¸à¸à¹à¹à¸à¹à¹à¸à¸à¸·à¹à¸à¸à¸µà¹à¸ªà¸µà¹à¸«à¸¥à¸·à¸­à¸ à¹à¸à¸·à¹à¸­ ยà¹à¸­/à¸à¸¢à¸²à¸¢ à¹à¸à¸¢à¹à¸à¹ RESTYLE: > >+à¸à¸¥à¸´à¸à¸à¸£à¸à¹à¸«à¸à¸à¹à¹à¸à¹à¹à¸à¸à¸·à¹à¸à¸à¸µà¹à¸ªà¸µà¹à¸«à¸¥à¸·à¸­à¸ à¹à¸à¸·à¹à¸­ ยà¹à¸­/à¸à¸¢à¸²à¸¢ : > > Did you check that it is appropriate to remove these words in these > languages? I tried to translate into English with Google translate and that seems to be correct.

(In reply to Karl Tomlinson (:karlt) from comment #5) > Comment on attachment 555085 [details] [diff] [review] > remove nonstandard actiontype "restyle" of maction on MathML demo pages. > > >-×××¥ ×¢× ××¢××ר ××× ×ק×× ×××××ר ×צ××× ××× ××××××/××ק××× ××¢×רת RESTYLE: > >+×××¥ ×¢× ××¢××ר ××× ×ק×× ×××××ר ×צ××× ××× ××××××/××ק×××: > > >-à¸à¸¥à¸´à¸à¸à¸£à¸à¹à¸«à¸à¸à¹à¹à¸à¹à¹à¸à¸à¸·à¹à¸à¸à¸µà¹à¸ªà¸µà¹à¸«à¸¥à¸·à¸­à¸ à¹à¸à¸·à¹à¸­ ยà¹à¸­/à¸à¸¢à¸²à¸¢ à¹à¸à¸¢à¹à¸à¹ RESTYLE: > >+à¸à¸¥à¸´à¸à¸à¸£à¸à¹à¸«à¸à¸à¹à¹à¸à¹à¹à¸à¸à¸·à¹à¸à¸à¸µà¹à¸ªà¸µà¹à¸«à¸¥à¸·à¸­à¸ à¹à¸à¸·à¹à¸­ ยà¹à¸­/à¸à¸¢à¸²à¸¢ : > > Did you check that it is appropriate to remove these words in these > languages? Exactly what fonts am I supposed to have installed so that this comment makes any sense whatsoever?

(In reply to Frédéric Wang (:fred) from comment #6) > I tried to translate into English with Google translate and that seems to be > correct. OK. Sounds fine, thanks. (In reply to Bill Gianopoulos from comment #7) > Exactly what fonts am I supposed to have installed so that this comment > makes any sense whatsoever? I don't think any fonts will. There seems to be an encoding (or similar) problem with Bugzilla's "Edit Attachment As Comment".

Where is the 2nd patch supposed to be landing, seeing as the files for it don't exist in m-c? In instances where non-standard landing of patches is required, please can you add suitable instructions to the whiteboard, to make it easier not only for the person doing the checkin-neededs, but also people doing the subsequent inbound -> m-c merge. Thanks :-)

Try run for the first patch: https://tbpl.mozilla.org/?tree=Try&usebuildbot=1&rev=5a17479d7da9

(In reply to Ed Morley [:edmorley] from comment #9) > Where is the 2nd patch supposed to be landing, seeing as the files for it > don't exist in m-c? > > In instances where non-standard landing of patches is required, please can > you add suitable instructions to the whiteboard, to make it easier not only > for the person doing the checkin-neededs, but also people doing the > subsequent inbound -> m-c merge. Thanks :-) I believe the second patch is for the www.mozilla.org webserver. to be landed relative to: http://www.mozilla.org/projects/mathml/ I suspect the checkin-needed was probably intended to refer to the first patch only.

Comment on attachment 555085 [details] [diff] [review] remove nonstandard actiontype "restyle" of maction on MathML demo pages. http://viewvc.svn.mozilla.org/vc?view=revision&revision=95002

Comment on attachment 555084 [details] [diff] [review] remove nonstandard actiontype "restyle" of maction. r=karlt (bug 398505) https://hg.mozilla.org/integration/mozilla-inbound/rev/c576f8acafd6

Documented: https://developer.mozilla.org/en/Firefox_9_for_developers#MathML https://developer.mozilla.org/en/MathML/Element/maction#Gecko-specific_notes