Closed Bug 29359 Opened 24 years ago Closed 24 years ago

Search.dll is loaded on startup

Categories

(Core :: XPCOM, defect, P3)

All
Windows NT
defect

Tracking

()

VERIFIED INVALID

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. ?
>
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
If this is a big startup slowdown (how much time are we losing, DP?), first 
thought: can we back out the code change?  ;^P
*** Bug 29363 has been marked as a duplicate of this bug. ***
I've marked the other bug (IE Favorites being loaded at startup) as a DUP of 
this bug, DP.
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
Nominating for beta1 based on timing reported. Remember robert is reporting 
timing on a real good powerbook.
Keywords: beta1
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-]
Status: NEW → ASSIGNED
Target Milestone: --- → M16
*** Bug 35789 has been marked as a duplicate of this bug. ***
adding perf kw..
Keywords: perf
*** Bug 35789 has been marked as a duplicate of this bug. ***
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
vrfy
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.