124226 - add timing code to native component loader

Status: RESOLVED WONTFIX

(Core :: XPCOM, defect)

Description

[Doug Turner (:dougt)]

Index: nsNativeComponentLoader.cpp
===================================================================
RCS file: /cvsroot/mozilla/xpcom/components/nsNativeComponentLoader.cpp,v
retrieving revision 1.77
diff -u -r1.77 nsNativeComponentLoader.cpp
--- nsNativeComponentLoader.cpp	29 Jan 2002 21:21:58 -0000	1.77
+++ nsNativeComponentLoader.cpp	7 Feb 2002 20:07:01 -0000
@@ -250,14 +250,16 @@
 {
 #ifdef DEBUG
     /* do we _really_ want to print this every time? */
+    PRIntervalTime startTime = PR_IntervalNow();
     printf("nsNativeComponentLoader: autoregistering begins.\n");
 #endif
 
     nsresult rv = RegisterComponentsInDir(aWhen, aDirectory);
 
 #ifdef DEBUG
-    printf("nsNativeComponentLoader: autoregistering %s\n",
-           NS_FAILED(rv) ? "FAILED" : "succeeded");
+    printf("nsNativeComponentLoader: autoregistering %s (%dms)\n",
+           NS_FAILED(rv) ? "FAILED" : "succeeded",
+           PR_IntervalToMilliseconds( PR_IntervalNow() - startTime ));
 #endif
 
     return rv;

Comment 1

[Doug Turner (:dougt)]

dp, this has been sitting in my tree for a while.  Can I get you to r= it?

Comment 2

[Suresh Duddi (gone)]

Sure. r=dp Since this is debug code, I dont think you should sweat on sr=

Comment 3

[Doug Turner (:dougt)]

not needed anytime soon.