83497 - Investigate adding namespace information to RuleHash

Status: VERIFIED FIXED

(Core :: CSS Parsing and Computation, defect, P2)

Description

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

We should investigate adding namespace information to RuleHash.  This would
improve performance (for the MathML rules shown in the logs on bug 83482 and for
the case that's only a problem because of bug 83495) by avoiding calling
SelectorMatches in some more cases.

I think it could work like this:  We could split the tag table into a separate
tag table and pseudo-tag table, and then make the keys for the tag table two
atoms instead of one.  We'd then have to match universal rules, universal
namespace rules (if the namespace is non-null), and tag rules.

Comment 1

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Of course it would be much easier to just have an mNamespaceTable.

Comment 2

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Fix checked in 2001-06-04 18:00 PDT.

Comment 3

[Hixie (not reading bugmail)]

Rubber stamp verification (code level bug, I saw the new code and it is indeed
adding namespce information to RuleHash as far as I could tell...).