Open Bug 1301881 Opened 8 years ago Updated 9 months ago

Investigate whether AddStateBits(NS_BLOCK_FLOAT_MGR) is needed in nsMathMLmathBlockFrame's constructor

Categories

(Core :: MathML, defect, P5)

Product:
Core
Component:
MathML
Type:
defect

Tracking

()

People

(Reporter: TYLin, Unassigned, NeedInfo)

References

Details

My refactor of NS_NewMathMLmathBlockFrame in bug 1301630 exposes a issue that AddStateBits(NS_BLOCK_FLOAT_MGR) set in nsMathMLmathBlockFrame's constructor is cleared in some cases. See bug 1301630 comment 5. We'd better investigate whether AddStateBits(NS_BLOCK_FLOAT_MGR) is still needed per bug 1301630 comment 6.
Per bug 1301630 comment 7 this was added initially for bug 353894. But the testcase from that bug doesn't crash or assert on trunk... Do we no longer need the float manager thing for line layout? It's not clear to me what the invariants around that stuff are nowadays.
Flags: needinfo?(dbaron)
Severity: normal → S3

https://hg.mozilla.org/mozilla-central/rev/76176850979d07028e80014ecbf6c51358c29b1b made NS_NewMathMLmathBlockFrame set that flag again and aFAIK that's the only way to create a nsMathMLmathBlockFrame.

You need to log in before you can comment on or make changes to this bug.