david is right, you'll need to do this for every tree instance.
my guess would be in doDelete() in bookmarks.js, just add
treeNode.treeBoxObject.beginBatch() and treeNode.treeBoxObject.endBatch()
to the right places. we're using 4 as our magic number for when to do
batch and when not to. I know it's a hack.
but wait until #66583 is fixed before you land anything that uses the
tree batching API.
http://bugzilla.mozilla.org/show_bug.cgi?id=66583
-Seth
David Bienvenu wrote:
> Yes, every tree instance, but it's probably only two lines of js per
> tree instance since your deletes are probably synchronous, I'm guessing.
>
> - david
>
> Chris McAfee wrote:
>
>> Randell Jesup wrote:
>>
>>> Seth Spitzer <[EMAIL PROTECTED]> writes:
>>> [batch api speedup]
>>>
>>> Any chance of getting this into Bookmarks as well?
>>
>>
>>
>> Yes! And history? Will this be a toolkit-level fix,
>> or do we need to fix/tweak each tree instance?
