nsMathMLmactionFrame::Init munges style context

RESOLVED FIXED in mozilla9

Status

()

Core
MathML
RESOLVED FIXED
10 years ago
6 years ago

People

(Reporter: bz, Assigned: fredw)

Tracking

(Blocks: 1 bug, {dev-doc-complete})

Trunk
mozilla9
x86
All
dev-doc-complete
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

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.
QA Contact: ian → mathml
(Assignee)

Updated

8 years ago
Blocks: 544036
(Assignee)

Updated

7 years ago
Assignee: rbs → nobody
(Assignee)

Comment 1

6 years ago
I suggest to remove the actiontype="restyle". See bug 544036 comment 1
Agreed.
(Assignee)

Comment 3

6 years ago
Created attachment 555084 [details] [diff] [review]
remove nonstandard actiontype "restyle" of maction. r=karlt (bug 398505)
Assignee: nobody → fred.wang
(Assignee)

Comment 4

6 years ago
Created attachment 555085 [details] [diff] [review]
remove nonstandard actiontype "restyle" of maction on MathML demo pages.
(Assignee)

Updated

6 years ago
Attachment #555084 - Flags: review?(karlt)
(Assignee)

Updated

6 years ago
Attachment #555085 - Flags: review?(karlt)
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?
Attachment #555085 - Flags: review?(karlt) → review+
Attachment #555084 - Flags: review?(karlt) → review+
(Assignee)

Comment 6

6 years ago
(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".
(Assignee)

Updated

6 years ago
Keywords: checkin-needed

Comment 9

6 years ago
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 :-)
Status: NEW → ASSIGNED
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
Attachment #555085 - Flags: checkin+
OS: Linux → All
Version: 1.8 Branch → Trunk
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
Attachment #555084 - Flags: checkin+

Updated

6 years ago
Keywords: checkin-needed
Target Milestone: --- → mozilla9
https://hg.mozilla.org/mozilla-central/rev/c576f8acafd6
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Documented:
https://developer.mozilla.org/en/Firefox_9_for_developers#MathML
https://developer.mozilla.org/en/MathML/Element/maction#Gecko-specific_notes
Keywords: dev-doc-complete
You need to log in before you can comment on or make changes to this bug.