140675 - Ts time increase with this checkin

Status: RESOLVED INVALID

(SeaMonkey :: General, defect, P1)

Description

[Chris McAfee]

This checkin from darin increased startup times on the
non-comet builds by 1% or so, this is a reminder to check
this out on Monday and make sure this time penalty is Ok.

http://bonsai.mozilla.org/cvsquery.cgi?module=MozillaTinderboxAll&branch=HEAD&cvsroot=/cvsroot&date=explicit&mindate=1019885220&maxdate=1019885820&who=darin%25netscape.com

Comment 1

[Darin Fisher]

ok, so i spent some time hacking on sleestack to see if i couldn't come up with
the solution to this regression.  i discovered the cause of at least half of the
problem, and i'm working on a fix now.

i have some ideas about what might be causing the other half of the Ts regression.
i'll try to investigate that tomorrow.

Comment 2

[Darin Fisher]

Attachment: preliminary patch (shaves off part of the Ts perf loss)

still needs more work

Comment 3

[Darin Fisher]

Attachment: preliminary patch (other helpful changes)

Comment 4

[Darin Fisher]

i'm working on a solution for this post-1.0

Comment 5

[Darin Fisher]

Attachment: yet another preliminary patch of goodies

Comment 6

[Darin Fisher]

ok, all of these patches are obsolete now.  in another bug i reverted all of the
startup code to use the native versions of the nsIFile methods.  hence, there's
no more work to do here.

marking INVALID.