Reviewed by Tim Hatcher

beidson · beidson · commit 99bf73cb77ca · 2006-08-25T21:28:33.000Z
Changed some debugging-only code to be more platform independent * WebCore.xcodeproj/project.pbxproj: * loader/icon/IconDatabase.cpp: (WebCore::IconDatabase::pruneUnretainedIconsOnStartup): (WebCore::IconDatabase::syncDatabase): Canonical link: https://commits.webkit.org/13437@main git-svn-id: https://svn.webkit.org/repository/webkit/trunk@16040 268f45cc-cd09-0410-ab3c-d52691b4dbfc
diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
@@ -1,3 +1,14 @@
+2006-08-25  Brady Eidson  <beidson@apple.com>
+
+        Reviewed by Tim Hatcher
+
+        Changed some debugging-only code to be more platform independent
+
+        * WebCore.xcodeproj/project.pbxproj:
+        * loader/icon/IconDatabase.cpp:
+        (WebCore::IconDatabase::pruneUnretainedIconsOnStartup):
+        (WebCore::IconDatabase::syncDatabase):
+
 2006-08-25  Nikolas Zimmermann  <zimmermann@kde.org>
 
         Reviewed/landed by Adam.
diff --git a/WebCore/loader/icon/IconDatabase.cpp b/WebCore/loader/icon/IconDatabase.cpp
@@ -30,13 +30,9 @@
 #include "Image.h"
 #include "Logging.h"
 #include "PlatformString.h"
-#include <errno.h>
-#include <sys/stat.h>
-#include <sys/types.h>
-#include <time.h>
 #include "SQLStatement.h"
 #include "SQLTransaction.h"
-
+#include "SystemTime.h"
 
 // FIXME - One optimization to be made when this is no longer in flux is to make query construction smarter - that is queries that are created from
 // multiple strings and numbers should be handled differently than with String + String + String + etc.
@@ -646,7 +642,7 @@ void IconDatabase::pruneUnretainedIconsOnStartup(Timer<IconDatabase>*)
     ASSERT(!m_initialPruningComplete);
 
 #ifndef NDEBUG
-    CFTimeInterval start = CFAbsoluteTimeGetCurrent();
+    double timestamp = currentTime();
 #endif
     
     // rdar://4690949 - Need to prune unretained iconURLs here, then prune out all pageURLs that reference
@@ -690,11 +686,11 @@ void IconDatabase::pruneUnretainedIconsOnStartup(Timer<IconDatabase>*)
     m_initialPruningComplete = true;
     
 #ifndef NDEBUG
-    CFTimeInterval duration = CFAbsoluteTimeGetCurrent() - start;
-    if (duration <= 1.0)
-        LOG(IconDatabase, "Pruning unretained icons took %.4f seconds", duration);
+    timestamp = currentTime() - timestamp;
+    if (timestamp <= 1.0)
+        LOG(IconDatabase, "Pruning unretained icons took %.4f seconds", timestamp);
     else
-        LOG(IconDatabase, "Pruning unretained icons took %.4f seconds - this is much too long!", duration);
+        LOG(IconDatabase, "Pruning unretained icons took %.4f seconds - this is much too long!", timestamp);
 #endif
 }
 
@@ -706,7 +702,7 @@ void IconDatabase::updateDatabase(Timer<IconDatabase>*)
 void IconDatabase::syncDatabase()
 {
 #ifndef NDEBUG
-    CFTimeInterval start = CFAbsoluteTimeGetCurrent();
+    double timestamp = currentTime();
 #endif
 
     // First we'll do the pending additions
@@ -751,11 +747,11 @@ void IconDatabase::syncDatabase()
     m_updateTimer.stop();
     
 #ifndef NDEBUG
-    CFTimeInterval duration = CFAbsoluteTimeGetCurrent() - start;
-    if (duration <= 1.0)
-        LOG(IconDatabase, "Updating the database took %.4f seconds", duration);
+    timestamp = currentTime() - timestamp;
+    if (timestamp <= 1.0)
+        LOG(IconDatabase, "Updating the database took %.4f seconds", timestamp);
     else 
-        LOG(IconDatabase, "Updating the database took %.4f seconds - this is much too long!", duration);
+        LOG(IconDatabase, "Updating the database took %.4f seconds - this is much too long!", timestamp);
 #endif
 }
 
