564576 - Speed up ChangeDocumentFor a bit

Status: RESOLVED DUPLICATE of bug 564591

(Core :: XBL, defect)

Description

[Boris Zbarsky [:bzbarsky]]

Attachment: Simple change

Ran into this while profiling the testcase in bug 564575.  The hashtable ops in ChangeDocumentFor hurt us.  I _think_ we can skip at least some of them.

I'm pretty sure about the SetAnonymousNodesFor being conditioned on |binding|.  What about the SetContentListFor change?  Is that ok?  The NODE_IS_INSERTION_PARENT flag is sadly underdocumented.  And do we need that SetInsertionParent call?  Either the node is being removed directly (and then ContentRemoved did the SetInsertionParent), or some ancestor is being removed, and then presumably the relevant bindings are being nuked and hence the insertion parent table will be updated correctly... right?

Comment 1

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

I have a better patch for this that seems to work. I've been whacking our BindToTree/UnbindFromTree code fairly hard. Will attach patches once the last of them are done.

Comment 2

[Boris Zbarsky [:bzbarsky]]

Oh?  I'd been glaring at our bind/unbind code, but not quite willing to spend enough time today to really take it apart.  Are your patches available somewhere so I can profile/test with them?  Is there a bug to follow?

Comment 3

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Bug 564591

Comment 4

[Boris Zbarsky [:bzbarsky]]

Indeed.