Changeset 154032 in webkit

Timestamp:

Aug 13, 2013 5:51:25 PM (11 years ago)

Author:

commit-queue@webkit.org

Message:

[WTF] [JSC] Replace currentTime() with monotonicallyIncreasingTime() in all possible places
​https://bugs.webkit.org/show_bug.cgi?id=119762

Patch by Arunprasad Rajkumar <​arurajku@cisco.com> on 2013-08-13
Reviewed by Geoffrey Garen.

Source/JavaScriptCore:

• heap/Heap.cpp:

(JSC::Heap::Heap):
(JSC::Heap::markRoots):
(JSC::Heap::collect):

• jsc.cpp:

(StopWatch::start):
(StopWatch::stop):

• testRegExp.cpp:

(StopWatch::start):
(StopWatch::stop):

Source/WTF:

• wtf/MainThread.cpp:

(WTF::dispatchFunctionsFromMainThread):

Location:

trunk/Source

Files:

• JavaScriptCore/ChangeLog (modified) (1 diff)
• JavaScriptCore/heap/Heap.cpp (modified) (6 diffs)
• JavaScriptCore/jsc.cpp (modified) (1 diff)
• JavaScriptCore/testRegExp.cpp (modified) (1 diff)
• WTF/ChangeLog (modified) (1 diff)
• WTF/wtf/MainThread.cpp (modified) (2 diffs)

trunk/Source/JavaScriptCore/ChangeLog

@@ +1 @@
+2013-08-13  Arunprasad Rajkumar  <arurajku@cisco.com>
+
+        [WTF] [JSC] Replace currentTime() with monotonicallyIncreasingTime() in all possible places
+        https://bugs.webkit.org/show_bug.cgi?id=119762
+
+        Reviewed by Geoffrey Garen.
+
+        * heap/Heap.cpp:
+        (JSC::Heap::Heap):
+        (JSC::Heap::markRoots):
+        (JSC::Heap::collect):
+        * jsc.cpp:
+        (StopWatch::start):
+        (StopWatch::stop):
+        * testRegExp.cpp:
+        (StopWatch::start):
+        (StopWatch::stop):
+
 2013-08-13  Julien Brianceau  <jbrianceau@nds.com>
 

trunk/Source/JavaScriptCore/heap/Heap.cpp

@@ -91 +91 @@
     GCTimerScope(GCTimer* timer)
         : m_timer(timer)
-        , m_start(WTF::currentTime())
+        , m_start(WTF::monotonicallyIncreasingTime())
     {
     }
     ~GCTimerScope()
     {
-        double delta = WTF::currentTime() - m_start;
+        double delta = WTF::monotonicallyIncreasingTime() - m_start;
         if (delta < m_timer->m_min)
             m_timer->m_min = delta;
@@ -264 +264 @@
     , m_vm(vm)
     , m_lastGCLength(0)
-    , m_lastCodeDiscardTime(WTF::currentTime())
+    , m_lastCodeDiscardTime(WTF::monotonicallyIncreasingTime())
     , m_activityCallback(DefaultGCActivityCallback::create(this))
     , m_sweeper(IncrementalSweeper::create(this))
@@ -434 +434 @@
 
 #if ENABLE(OBJECT_MARK_LOGGING)
-    double gcStartTime = WTF::currentTime();
+    double gcStartTime = WTF::monotonicallyIncreasingTime();
 #endif
 
@@ -591 +591 @@
     visitCount += m_sharedData.childVisitCount();
 #endif
-    MARK_LOG_MESSAGE2("\nNumber of live Objects after full GC %lu, took %.6f secs\n", visitCount, WTF::currentTime() - gcStartTime);
+    MARK_LOG_MESSAGE2("\nNumber of live Objects after full GC %lu, took %.6f secs\n", visitCount, WTF::monotonicallyIncreasingTime() - gcStartTime);
 #endif
 
@@ -727 +727 @@
     m_activityCallback->willCollect();
 
-    double lastGCStartTime = WTF::currentTime();
+    double lastGCStartTime = WTF::monotonicallyIncreasingTime();
     if (lastGCStartTime - m_lastCodeDiscardTime > minute) {
         deleteAllCompiledCode();
-        m_lastCodeDiscardTime = WTF::currentTime();
+        m_lastCodeDiscardTime = WTF::monotonicallyIncreasingTime();
     }
 
@@ -798 +798 @@
 
     m_bytesAllocated = 0;
-    double lastGCEndTime = WTF::currentTime();
+    double lastGCEndTime = WTF::monotonicallyIncreasingTime();
     m_lastGCLength = lastGCEndTime - lastGCStartTime;
 

trunk/Source/JavaScriptCore/jsc.cpp

@@ -172 +172 @@
 void StopWatch::start()
 {
-    m_startTime = currentTime();
+    m_startTime = monotonicallyIncreasingTime();
 }
 
 void StopWatch::stop()
 {
-    m_stopTime = currentTime();
+    m_stopTime = monotonicallyIncreasingTime();
 }
 

trunk/Source/JavaScriptCore/testRegExp.cpp

@@ -83 +83 @@
 void StopWatch::start()
 {
-    m_startTime = currentTime();
+    m_startTime = monotonicallyIncreasingTime();
 }
 
 void StopWatch::stop()
 {
-    m_stopTime = currentTime();
+    m_stopTime = monotonicallyIncreasingTime();
 }
 

trunk/Source/WTF/ChangeLog

@@ +1 @@
+2013-08-13  Arunprasad Rajkumar  <arurajku@cisco.com>
+
+        [WTF] [JSC] Replace currentTime() with monotonicallyIncreasingTime() in all possible places
+        https://bugs.webkit.org/show_bug.cgi?id=119762
+
+        Reviewed by Geoffrey Garen.
+
+        * wtf/MainThread.cpp:
+        (WTF::dispatchFunctionsFromMainThread):
+
 2013-08-12  Ragner Magalhaes  <ranger.n@samsung.com>
 

trunk/Source/WTF/wtf/MainThread.cpp

@@ -142 +142 @@
         return;
 
-    double startTime = currentTime();
+    double startTime = monotonicallyIncreasingTime();
 
     FunctionWithContext invocation;
@@ -163 +163 @@
         // This code has effect only in case the scheduleDispatchFunctionsOnMainThread() is implemented in a way that
         // allows input events to be processed before we are back here.
-        if (currentTime() - startTime > maxRunLoopSuspensionTime) {
+        if (monotonicallyIncreasingTime() - startTime > maxRunLoopSuspensionTime) {
             scheduleDispatchFunctionsOnMainThread();
             break;