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

RESOLVED FIXED in mozilla29

Status

()

RESOLVED FIXED
7 years ago
5 years ago

People

(Reporter: jruderman, Assigned: fredw)

Tracking

(Blocks: 1 bug, {assertion, testcase})

Trunk
mozilla29
x86_64
macOS
assertion, testcase
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [mentor=fredw][lang=c++])

Attachments

(2 attachments)

(Reporter)

Description

7 years ago
Posted file 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
(Reporter)

Comment 1

7 years ago
Posted file stack trace
(Assignee)

Comment 2

7 years ago
This assertion was added in bug 534963.
(Assignee)

Comment 3

6 years ago
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.
Whiteboard: [mentor=fredw][lang=c++]
(Assignee)

Updated

6 years ago
Depends on: 838506
(Assignee)

Comment 4

5 years ago
Fixed per bug 838506.
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Assignee: nobody → fred.wang
Target Milestone: --- → mozilla29
You need to log in before you can comment on or make changes to this bug.