Closed
Bug 29359
Opened 24 years ago
Closed 24 years ago
Search.dll is loaded on startup
Categories
(Core :: XPCOM, defect, P3)
Tracking
()
VERIFIED
INVALID
M16
People
(Reporter: dp, Assigned: waterson)
References
Details
(Keywords: perf, Whiteboard: [PDT-])
The search dll is being loaded on startup. This is xp. I see this on windows and unix. Rjc is seeing it on mac. Subject: Re: libsearch.so loaded on startup Date: Fri, 25 Feb 2000 17:34:42 -0800 From: waterson@netscape.com (Chris Waterson) To: rjc@netscape.com CC: Suresh Duddi <dp@netscape.com> References: 1 , 2 Is this XP? Or Mac-only? My guess is that this is due to my change from about a week ago to build whatever frames layout asked for. chris Robert Churchill wrote: > > Hey guys, > > It looks like we *are* now reading in IE Favorites (as well as all the > search datasets) on startup, instead of deferring like we used to. > > At first glance, it looks like nsXULDocument is being really eager in > terms of building up RDF-generated menu content. > > Chris, it looks like you might know more about this from the CVS > logs. :^) What's the purpose of ResumeWalk() et.al. ? >
Reporter | ||
Comment 1•24 years ago
|
||
Robert, I am assigning this to you hoping you would help waterson solve this. He already has a few pdt+ and a ton of mozilla+ issues to deal with :-)
Blocks: 29249
Comment 2•24 years ago
|
||
If this is a big startup slowdown (how much time are we losing, DP?), first thought: can we back out the code change? ;^P
Comment 4•24 years ago
|
||
I've marked the other bug (IE Favorites being loaded at startup) as a DUP of this bug, DP.
Comment 5•24 years ago
|
||
OK, I'm whimping out on this and giving this bug to Chris so see what he wants to do regarding this. (Fix/back-out/ignore) The biggest hit is on the Mac of course, as it has to: 1) read in favorites.html (On Windows, nothing needs to be read in, as IE Favorites are stored in the file system) 2) read in all the Sherlock files (not just the ones we ship with, but with all the files including with the OS as well) Item #1 appears to be about 1/10th of a second on my Powerbook. Item #2 appears to be 3 or 4 seconds (yikes!) on my Powerbook.
Assignee: rjc → waterson
Reporter | ||
Comment 6•24 years ago
|
||
Nominating for beta1 based on timing reported. Remember robert is reporting timing on a real good powerbook.
Keywords: beta1
Comment 7•24 years ago
|
||
Please indicate if there is an easy fix to delay the loading. Even better would be a fix in hand, but a plan for an easy fix would be very attractive. Thanks, Jim
Putting on PDT- for beta1. Please put make on PDT radar when you'all have a plan :-)
Whiteboard: [PDT-]
Assignee | ||
Updated•24 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Updated•24 years ago
|
Target Milestone: --- → M16
Comment 11•24 years ago
|
||
*** Bug 35789 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 12•24 years ago
|
||
search.dll is loaded on startup because it's included in the "datasources=" tag for both the personal toolbar and the bookmarks menu. I've discussed with rjc, and he feels that we should leave it there. Marking as INVALID for this reason. FWIW, I've set a breakpoint in InternetSearchService::DeferredInit(), which should be called when the first search is issued. I'm never seeing it hit.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → INVALID
You need to log in
before you can comment on or make changes to this bug.
Description
•