391836 - slow to copy lots of nested bookmarks

Status: RESOLVED DUPLICATE of bug 472343

(Firefox :: Bookmarks & History, defect, P3)

Description

[(not reading, please use [e-mail] instead)]

slow to copy lots of nested bookmarks

steps to reproduce:

1) have a profile with lots of bookmarks with subfolders
2) open bm organizer, select all, copy

you will get the busy cursor and firefox will not be responsive.

I am seeing is lots of command updating.  updating could be happening as we expand and collapse folders recursively when we wrap nodes?

Comment 1

[(not reading, please use [e-mail] instead)]

> I am seeing is lots of command updating.

boris recently logged a bug about command updating, and mano fixed it (see bug #394695)

Mano may have fixed this bug as well, it would be worth re-testing.

Comment 2

[Dietrich Ayala (:dietrich)]

Lots of perf improvements to the wrap/unwrap code in bug 384370, may mitigate this bug.

Comment 3

[Asaf Romano (gone)]

Seth: is this still reproducible?

Comment 4

[Marco Bonardo [:mak]]

i've seen this recently so i think it's still reproducible, when doing copy/paste or draggin a big folder with many entries and subfolders we are still slow. But IIRC there's also another bug on this

Comment 5

[Alex Riesen]

Me too. I don't even have to select all of my bookmarks - one folder is enough to load this P4/2.6GHz for a minute.

X11; U; Linux i686; en-US; rv:1.9.0.4 Gecko/2008111318 Ubuntu/8.10 (intrepid) Firefox/3.0.4

Comment 6

[Gervase Markham [:gerv]]

Bug 451915 - move Firefox/Places bugs to Firefox/Bookmarks and History. Remove all bugspam from this move by filtering for the string "places-to-b-and-h".

In Thunderbird 3.0b, you do that as follows:
Tools | Message Filters
Make sure the correct account is selected. Click "New"
Conditions: Body   contains   places-to-b-and-h
Change the action to "Delete Message".
Select "Manually Run" from the dropdown at the top.
Click OK.

Select the filter in the list, make sure "Inbox" is selected at the bottom, and click "Run Now". This should delete all the bugspam. You can then delete the filter.

Gerv

Comment 7

[Dimitrios]

Sorry for spamming but are you still seeing this, after bug 472343 was fixed?

Comment 8

[Marco Bonardo [:mak]]

Let's call this a dupe, the code changed pretty much and it's hard to link this bug to the original cause.
We have other bugs handling missing perf gains.