15752 - Registry file opening should use nsIFile

Status: RESOLVED WONTFIX

Description

[Simon Fraser]

For performance reasons, we should use an nsFileSpec when opening the components
registry file (and other registry files) if at all possible. On Mac, the
nsFileSpec to full path conversion takes around 14ms on a fast machine,
which accounts for 6% of the time spent in NS_InitXPCOM. There will be another
slowdown in the standard library when the full path is converted back to an
FSSpec to open the file.

Comment 1

[Daniel Veditz [:dveditz]]

Could this wait until after beta? We'll want to make some speed improvements
then, too, right? :-)

Comment 2

[Simon Fraser]

I fear that items that get pushed to 'after beta' will be dropped because of
schedule pressure. This isn't a huge performance hit, so it could wait. But if
you are doing any other related registry work, you might consider this too.

Comment 3

[Daniel Veditz [:dveditz]]

Needs to be Beta

Comment 4

[Daniel Veditz [:dveditz]]

Need to convert the registry over to the new nsIFile.

Comment 5

[Daniel Veditz [:dveditz]]

Making pdt+ bugs P1

Comment 6

[leger]

Putting on PDT- radar for beta1.  This is not a huge perf hit.  Other areas more 
critical.  Would not hold beta for this.

Comment 7

[Daniel Veditz [:dveditz]]

pdt-, pushing to M15

Comment 8

[Daniel Veditz [:dveditz]]

Taking advantage of new "Future" target milestone.

Comment 9

[Dawn Endico]

dp is no longer @netscape.com. reassigning qa contact to default for this component

Comment 10

[Daniel Veditz [:dveditz]]

Maybe next time

Comment 11

[Pete Collins (MDG)]

Hasn't this been bug fixed by dougt lately??

--pete

Comment 12

[Simon Paquet [:sipaq]]

This bug is targeted at a Mac classic platform/OS, which is no longer supported
by mozilla.org. Please re-target it to another platform/OS if this bug applies
there as well or resolve this bug.

I will resolve this bug as WONTFIX in four weeks if no action has been taken.
To filter this and similar messages out, please filter for "mac_cla_reorg".

Comment 13

[Simon Paquet [:sipaq]]

This bug hasen't been touched for years and is clearly unowned. Moving back to default assignee/QA so that people, who are watching those can accurately triage this bug.

Also resetting Priority, Target Milestone, Status Whiteboard and Status.