1338723 - Free up node bit by encoding directionality state more compactly

Status: RESOLVED WONTFIX

(Core :: DOM: Core & HTML, defect, P3)

Description

[Bobby Holley (:bholley)]

See bug 1331322 comment 10.

Comment 1

[Andrew Overholt [:overholt]]

This is an optimization that's not super-critical, right, Bobby?

Comment 2

[Bobby Holley (:bholley)]

(In reply to Andrew Overholt [:overholt] from comment #1)
> This is an optimization that's not super-critical, right, Bobby?

It is not critical. In fact, it's not really worth doing until the next time somebody needs a node bit (modulo the fact that people may not be aware of the potential to free this one up).

Comment 3

[Olli Pettay [:smaug][[e-mail]]]

There was recently need for a bit: IntersectionObservers wouldn't need to increase the size of already too large slots if they could have been stored similarly to EventListenerManagers

Comment 4

[Boris Zbarsky [:bzbarsky]]

Right.  This is not critical until someone needs a bit.  ;)  Then it's really good to know we can do this.

Comment 5

[Boris Zbarsky [:bzbarsky]]

Bug 1373798 would be an alternative fix for this.

Comment 6

[Boris Zbarsky [:bzbarsky]]

Wontfix, since now we only use one node bit here.