606925 - Refresh the HTML5 parser to spec on insertion mode resetting

Status: RESOLVED FIXED

(Core :: DOM: HTML Parser, defect, P2)

Description

[Henri Sivonen (:hsivonen)]

Apparently the spec has changed on how <math><mi><div><object><div><span></span></div></object></div></mi><mi> should parse. Need to track down the spec change and fix the parser accordingly.

Comment 1

[Henri Sivonen (:hsivonen)]

Per discussion with jst, nominating all HTML5 parsing spec change bugs as
blockers.

This bug is part of the family of bugs arising from spec-level optimizations around foreign content. Per discussion with jst, I'm planning on fixing this by de-optimizing the implementation to implement the conceptual model instead of the spec's imperfect optimization of the conceptual model.

The main reason why this should be fixed for Firefox 4 is that it's confusing to authors if foreign content doesn't work intuitively and there are bizarre edge cases.

Comment 2

[Johnny Stenback (:jst)]

Yup, I think we should block on this.

Comment 3

[Henri Sivonen (:hsivonen)]

Attachment: Get rid of the "in foreign content" insertion mode. Check the namespace of the current node instead.

As mentioned in an earlier comment, this fix implements the conceptual model of behind the spec directly instead of implementing the more complex spec-level optimizations.

Comment 4

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Comment on attachment 493249 [details] [diff] [review]
Get rid of the "in foreign content" insertion mode. Check the namespace of the current node instead.

rs=me

Comment 5

[Henri Sivonen (:hsivonen)]

http://hg.mozilla.org/mozilla-central/rev/a6ad532d8bf8