59 bytes, text/x-review-board-request
No description provided.
Comment on attachment 8964798 [details] Bug 1451239: Remove resolved namespace id and tag from FCItems. https://reviewboard.mozilla.org/r/233526/#review239928 r=me ::: commit-message-96cea:7 (Diff revision 1) > + > +Tag is unused. > + > +This changes how some mixes of MathML and html get wrapped in anonymous table > +boxes (in particular, it changes whether it uses a MathML or an HTML table > +frame). So the main thing this affects is whether the frame responds to certain attributes. That said, responding to mathml attributes on its mContent when that mContent is not a MathML element seems ... weird. So arguably the new setup is just more correct. Might be worth it to explain that. ::: layout/base/nsCSSFrameConstructor.cpp:5530 (Diff revision 1) > "Unexpected display"); > > static const FrameConstructionData sPageBreakData = > FCDATA_DECL(FCDATA_SKIP_FRAMESET, NS_NewPageBreakFrame); > > // Lie about the tag and namespace so we don't trigger anything We're no longer lying about anything here.
Attachment #8964798 - Flags: review?(bzbarsky) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/9402bfb8bc8d Remove resolved namespace id and tag from FCItems. r=bz
You need to log in before you can comment on or make changes to this bug.