713606 - "ASSERTION: aWhichFlags should only be displaystyle or compression flag"

Status: RESOLVED FIXED

(Core :: MathML, defect)

Description

[Jesse Ruderman]

Attachment: testcase

###!!! ASSERTION: aWhichFlags should only be displaystyle or compression flag: 'NS_MATHML_IS_DISPLAYSTYLE(aWhichFlags) || NS_MATHML_IS_COMPRESSED(aWhichFlags)', file layout/mathml/nsMathMLFrame.cpp, line 160

Comment 1

[Jesse Ruderman]

Attachment: stack trace

Comment 2

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

This assertion was added in bug 534963.

Comment 3

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

Here are another way to get the assertion:

   <math>
      <msubsup>
        <mtext>X</mtext>
        <mtable displaystyle="true"/>
	</mtext>Y</mtext>
      </msubsup>
    </math>

(or same with mtable replaced by mstyle). 

nsMathMLmtableOuterFrame::UpdatePresentationData and nsMathMLmstyleFrame::UpdatePresentationData intentionally disable the flag for displaystyle update and then call the their ancestor UpdatePresentationData, possibly to update the compression flag. So I'm not sure adding the assertion was quite correct or otherwise we would have to verify in each UpdatePresentationData that disable the displaystyle flag that the compression flag is set before calling the ancestor UpdatePresentationData...

I suspect fixing bug 838506 would also solve that issue.

Comment 4

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

Fixed per bug 838506.