Closed Bug 83497 Opened 25 years ago Closed 25 years ago

Investigate adding namespace information to RuleHash

Categories

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

Product:
Core
Component:
CSS Parsing and Computation
Type:
defect

Tracking

()

Status:
VERIFIED FIXED
Milestone:
mozilla0.9.2

People

(Reporter: dbaron, Assigned: dbaron)

References

Details

(Keywords: perf)

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.
Status: NEW → ASSIGNED
Priority: -- → P2
Target Milestone: --- → mozilla0.9.2
Of course it would be much easier to just have an mNamespaceTable.
Fix checked in 2001-06-04 18:00 PDT.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
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...).
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.