Closed Bug 398896 Opened 16 years ago Closed 16 years ago

New history search hang since 6 Sep 2007


(Firefox :: Bookmarks & History, defect)

Windows XP
Not set



Firefox 3 beta1


(Reporter: ria.klaassen, Assigned: dietrich)



(Keywords: hang, perf, regression)


(1 file)

Steps to reproduce:

- Create a new profile and paste a places.sqlite in it (if it doesn't succeed with your own places.sqlite I have a specially created example that could be used for this goal)
- Open history sidebar (use default View setting = By Date) and do a search for the word "bonsai". There should be 4 entries in it

Behaviour before 6 Sep: it finds the 4 entries within 1 or 2 seconds.
Current behaviour: it hangs. After one minute I force closed Firefox.

Regression range for this is 
So I think it is caused by by bug 393464.
A test on my husband's vista computer proved that the bug also occurs on faster systems, but less obvious (7 seconds, CPU 75%).
Ria, this was backed out for 5 days, can you see if you can reproduce during that period?

1.170	2007-09-15 12:53	 	Bug 393464 - Re-landing tag-search so search-in-folders (bug 384228) and related work could get the same sort of love. patch by dietrich, r=sspitzer, a=mconnor.

1.169	2007-09-10 11:08	 	Bug 393464 Bookmarks manager doesn't search in tags field (reverting)
I tested a 11 Sep 2007 build and indeed, no problem. 
ok Ria, thanks for checking.
Assignee: nobody → dietrich
Flags: blocking-firefox3?
Target Milestone: --- → Firefox 3 M9
Attached patch fixSplinter Review
go directly to db. nsNavHistory::URIHasTag() went from 4-6ms to <1ms in my tests. this change is covered by the unit tests for bug 395101, and those still pass w/ this patch.
Attachment #283965 - Flags: review?(sspitzer)
Flags: blocking-firefox3? → blocking-firefox3+
Comment on attachment 283965 [details] [diff] [review]

Attachment #283965 - Flags: review?(sspitzer) → review+
Checking in toolkit/components/places/src/nsNavHistory.cpp;
/cvsroot/mozilla/toolkit/components/places/src/nsNavHistory.cpp,v  <--  nsNavHistory.cpp
new revision: 1.176; previous revision: 1.175
Checking in toolkit/components/places/src/nsNavHistory.h;
/cvsroot/mozilla/toolkit/components/places/src/nsNavHistory.h,v  <--  nsNavHistory.h
new revision: 1.106; previous revision: 1.105
Closed: 16 years ago
Resolution: --- → FIXED
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.

Component: Places → Bookmarks & History
QA Contact: places → bookmarks
You need to log in before you can comment on or make changes to this bug.