Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1276)

Unified Diff: Source/platform/Timer.h

Issue 1134523002: Implement timers by posting delayed tasks (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Rebased. Created 5 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/platform/ThreadTimers.cpp ('k') | Source/platform/Timer.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/platform/Timer.h
diff --git a/Source/platform/Timer.h b/Source/platform/Timer.h
index fcc01843617f71c3eff8261d914d306ceb1195d6..38d3dc167af334cbdd73cf6ca09f826ace4cab00 100644
--- a/Source/platform/Timer.h
+++ b/Source/platform/Timer.h
@@ -28,6 +28,7 @@
#include "platform/PlatformExport.h"
#include "platform/heap/Handle.h"
+#include "platform/scheduler/CancellableTaskFactory.h"
#include "public/platform/WebTraceLocation.h"
#include "wtf/AddressSanitizer.h"
#include "wtf/Noncopyable.h"
@@ -65,11 +66,12 @@ public:
double repeatInterval() const { return m_repeatInterval; }
void augmentRepeatInterval(double delta) {
- setNextFireTime(m_nextFireTime + delta);
+ double now = monotonicallyIncreasingTime();
+ setNextFireTime(now, m_nextFireTime - now + delta);
m_repeatInterval += delta;
}
- void didChangeAlignmentInterval();
+ void didChangeAlignmentInterval(double now);
private:
virtual void fired() = 0;
@@ -77,35 +79,16 @@ private:
NO_LAZY_SWEEP_SANITIZE_ADDRESS
virtual double alignedFireTime(double fireTime) const { return fireTime; }
- void checkConsistency() const;
- void checkHeapIndex() const;
+ void setNextFireTime(double now, double delay);
- void setNextFireTime(double);
-
- NO_LAZY_SWEEP_SANITIZE_ADDRESS
- bool inHeap() const { return m_heapIndex != -1; }
-
- bool hasValidHeapPosition() const;
- void updateHeapIfNeeded(double oldTime);
-
- void heapDecreaseKey();
- void heapDelete();
- void heapDeleteMin();
- void heapIncreaseKey();
- void heapInsert();
- void heapPop();
- void heapPopMin();
-
- NO_LAZY_SWEEP_SANITIZE_ADDRESS
- Vector<TimerBase*>& timerHeap() const { ASSERT(m_cachedThreadGlobalTimerHeap); return *m_cachedThreadGlobalTimerHeap; }
+ void run();
double m_nextFireTime; // 0 if inactive
double m_unalignedNextFireTime; // m_nextFireTime not considering alignment interval
double m_repeatInterval; // 0 if not repeating
- int m_heapIndex; // -1 if not in heap
- unsigned m_heapInsertionOrder; // Used to keep order among equal-fire-time timers
- Vector<TimerBase*>* m_cachedThreadGlobalTimerHeap;
WebTraceLocation m_location;
+ CancellableTaskFactory m_cancellableTaskFactory;
+ WebScheduler* m_webScheduler; // Not owned.
#if ENABLE(ASSERT)
ThreadIdentifier m_thread;
@@ -164,7 +147,7 @@ NO_LAZY_SWEEP_SANITIZE_ADDRESS
inline bool TimerBase::isActive() const
{
ASSERT(m_thread == currentThread());
- return m_nextFireTime;
+ return m_cancellableTaskFactory.isPending();
}
}
« no previous file with comments | « Source/platform/ThreadTimers.cpp ('k') | Source/platform/Timer.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698