Re: Review of Bi-Di code, bug 62777

[Erik van der Poel]

Lina Kemel wrote:
> 
> The problem is that we have to know at the same time: 1) tag/node info
> (BDO) and 2) attribute (DIR). One place where those are available, is
> nsPresContext::ResolveStyleContextFor, after style resolution. But then
> 2 additional includes are needed:
> nsINameSpaceManager.h  // for kNameSpaceID_HTML
> nsHTMLAtoms.h          // for nsHTMLAtoms::dir

That doesn't seem like the right place to look for the BDO element and
DIR attribute. It would probably be better to do it where other HTML
elements and attributes are "mapped to style". I forgot where that was,
but I'm sure there are others in this newsgroup that know where it is.
It is important to map this correctly, because there are certain rules
about precedence when both HTML elements/attributes and CSS properties
are fighting over the same piece of text.

Erik