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

RESOLVED FIXED in mozilla29

Status

()

Core
MathML
RESOLVED FIXED
5 years ago
3 years ago

People

(Reporter: Jesse Ruderman, Assigned: fredw)

Tracking

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

Trunk
mozilla29
x86_64
Mac OS X
assertion, testcase
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

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

Attachments

(2 attachments)

(Reporter)

Description

5 years ago
Created attachment 584386 [details]
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

5 years ago
Created attachment 584387 [details]
stack trace
(Assignee)

Comment 2

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

Comment 3

4 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

4 years ago
Depends on: 838506
(Assignee)

Comment 4

3 years ago
Fixed per bug 838506.
Status: NEW → RESOLVED
Last Resolved: 3 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.