Changeset 41456 in webkit

Timestamp:

Mar 5, 2009 11:58:10 AM (15 years ago)

Author:

treat@webkit.org

Message:

2009-03-05 Yong Li <​yong.li@torchmobile.com>

Reviewed by Antti Koivisto.

​https://bugs.webkit.org/show_bug.cgi?id=24392
Do not get the current time for unless we're doing PRELOAD_DEBUG as this
can be unnecessarily expensive.

• html/PreloadScanner.cpp: (WebCore::PreloadScanner::write):

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• html/PreloadScanner.cpp (modified) (2 diffs)

trunk/WebCore/ChangeLog

@@ +1 @@
+2009-03-05  Yong Li  <yong.li@torchmobile.com>
+
+        Reviewed by Antti Koivisto.
+
+        https://bugs.webkit.org/show_bug.cgi?id=24392
+        Do not get the current time for unless we're doing PRELOAD_DEBUG as this
+        can be unnecessarily expensive.
+
+        * html/PreloadScanner.cpp:
+        (WebCore::PreloadScanner::write):
+
 2009-03-04  Simon Fraser  <simon.fraser@apple.com>
 

trunk/WebCore/html/PreloadScanner.cpp

@@ -1 +1 @@
 /*
  * Copyright (C) 2008 Apple Inc. All Rights Reserved.
+ * Copyright (C) 2009 Torch Mobile, Inc. http://www.torchmobile.com/
  *
  * Redistribution and use in source and binary forms, with or without
@@ -129 +130 @@
 void PreloadScanner::write(const SegmentedString& source)
 {
+#if PRELOAD_DEBUG
     double startTime = currentTime();
+#endif
     tokenize(source);
+#if PRELOAD_DEBUG
     m_timeUsed += currentTime() - startTime;
+#endif
 }