651103 - Editor should not rely on the legacy parser for IsBlock and IsContainer

Status: NEW

(Core :: DOM: Editor, defect, P5)

Description

[Henri Sivonen (:hsivonen)]

Currently, Editor uses nsParserService to query elements for blockness or containerness. nsParserService relies on nsElementTable, which is part of the old parser and should go away. Editor should get some new source of information for this so that the bulk of nsElementTable can go away.

Comment 1

[(no longer active)]

(In reply to comment #0)
> Currently, Editor uses nsParserService to query elements for blockness or
> containerness. nsParserService relies on nsElementTable, which is part of the
> old parser and should go away. Editor should get some new source of information
> for this so that the bulk of nsElementTable can go away.

Do you have a suggestion for this new source?

Comment 2

[Henri Sivonen (:hsivonen)]

(In reply to comment #1)
> Do you have a suggestion for this new source?

I'd put 
virtual PRBool IsContainer() {
  return PR_FALSE;
}
virtual PRBool IsBlock() {
  return PR_FALSE;
}
on nsIContent and override those in the per-element subclasses as appropriate according to the WHATWG HTML spec.

Comment 3

[Jens Stutte [:jstutte]]

Bulk-downgrade of unassigned, >=5 years untouched DOM/Storage bugs' priority and severity.

If you have reason to believe this is wrong, please write a comment and ni :jstutte.