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

Unified Diff: cc/scheduler/delay_based_time_source.h

Issue 12623026: cc: Chromify TimeSource, DelayBasedTimeSource and test (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 9 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 | « no previous file | cc/scheduler/delay_based_time_source.cc » ('j') | cc/scheduler/delay_based_time_source.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/scheduler/delay_based_time_source.h
diff --git a/cc/scheduler/delay_based_time_source.h b/cc/scheduler/delay_based_time_source.h
index d2e2a78aff2bf7f2ab6bfab9577c5247f40bc296..06a4ee700e7ae26f11f71d8fb85e7943a705f964 100644
--- a/cc/scheduler/delay_based_time_source.h
+++ b/cc/scheduler/delay_based_time_source.h
@@ -14,66 +14,67 @@ namespace cc {
class Thread;
// This timer implements a time source that achieves the specified interval
-// in face of millisecond-precision delayed callbacks and random queueing delays.
+// in face of millisecond-precision delayed callbacks and random queueing
+// delays.
class CC_EXPORT DelayBasedTimeSource : public TimeSource {
-public:
- static scoped_refptr<DelayBasedTimeSource> create(base::TimeDelta interval, Thread* thread);
-
- virtual void setClient(TimeSourceClient* client) OVERRIDE;
-
- // TimeSource implementation
- virtual void setTimebaseAndInterval(base::TimeTicks timebase, base::TimeDelta interval) OVERRIDE;
-
- virtual void setActive(bool) OVERRIDE;
- virtual bool active() const OVERRIDE;
-
- // Get the last and next tick times. nextTimeTime() returns null when
- // inactive.
- virtual base::TimeTicks lastTickTime() OVERRIDE;
- virtual base::TimeTicks nextTickTime() OVERRIDE;
-
-
- // Virtual for testing.
- virtual base::TimeTicks now() const;
-
-protected:
- DelayBasedTimeSource(base::TimeDelta interval, Thread* thread);
- virtual ~DelayBasedTimeSource();
-
- base::TimeTicks nextTickTarget(base::TimeTicks now);
- void postNextTickTask(base::TimeTicks now);
- void onTimerFired();
-
- enum State {
- STATE_INACTIVE,
- STATE_STARTING,
- STATE_ACTIVE,
- };
-
- struct Parameters {
- Parameters(base::TimeDelta interval, base::TimeTicks tickTarget)
- : interval(interval), tickTarget(tickTarget)
- { }
- base::TimeDelta interval;
- base::TimeTicks tickTarget;
- };
-
- TimeSourceClient* m_client;
- bool m_hasTickTarget;
- base::TimeTicks m_lastTickTime;
-
- // m_currentParameters should only be written by postNextTickTask.
- // m_nextParameters will take effect on the next call to postNextTickTask.
- // Maintaining a pending set of parameters allows nextTickTime() to always
- // reflect the actual time we expect onTimerFired to be called.
- Parameters m_currentParameters;
- Parameters m_nextParameters;
-
- State m_state;
-
- Thread* m_thread;
- base::WeakPtrFactory<DelayBasedTimeSource> m_weakFactory;
- DISALLOW_COPY_AND_ASSIGN(DelayBasedTimeSource);
+ public:
+ static scoped_refptr<DelayBasedTimeSource> Create(base::TimeDelta interval,
+ Thread* thread);
+
+ virtual void SetClient(TimeSourceClient* client) OVERRIDE;
+
+ // TimeSource implementation
+ virtual void SetTimebaseAndInterval(base::TimeTicks timebase,
+ base::TimeDelta interval) OVERRIDE;
+
+ virtual void SetActive(bool active) OVERRIDE;
+ virtual bool Active() const OVERRIDE;
+
+ // Get the last and next tick times. nextTimeTime() returns null when
+ // inactive.
+ virtual base::TimeTicks LastTickTime() OVERRIDE;
+ virtual base::TimeTicks NextTickTime() OVERRIDE;
+
+ // Virtual for testing.
+ virtual base::TimeTicks Now() const;
+
+ protected:
+ DelayBasedTimeSource(base::TimeDelta interval, Thread* thread);
+ virtual ~DelayBasedTimeSource();
+
+ base::TimeTicks NextTickTarget(base::TimeTicks now);
+ void PostNextTickTask(base::TimeTicks now);
+ void OnTimerFired();
+
+ enum State {
+ STATE_INACTIVE,
+ STATE_STARTING,
+ STATE_ACTIVE,
+ };
+
+ struct Parameters {
+ Parameters(base::TimeDelta interval, base::TimeTicks tick_target)
+ : interval(interval), tick_target(tick_target) {}
+ base::TimeDelta interval;
+ base::TimeTicks tick_target;
+ };
+
+ TimeSourceClient* client_;
+ bool has_tick_target_;
+ base::TimeTicks last_tick_time_;
+
+ // current_parameters_ should only be written by PostNextTickTask.
+ // next_parameters_ will take effect on the next call to PostNextTickTask.
+ // Maintaining a pending set of parameters allows NextTickTime() to always
+ // reflect the actual time we expect OnTimerFired to be called.
+ Parameters current_parameters_;
+ Parameters next_parameters_;
+
+ State state_;
+
+ Thread* thread_;
+ base::WeakPtrFactory<DelayBasedTimeSource> weak_factory_;
+ DISALLOW_COPY_AND_ASSIGN(DelayBasedTimeSource);
};
} // namespace cc
« no previous file with comments | « no previous file | cc/scheduler/delay_based_time_source.cc » ('j') | cc/scheduler/delay_based_time_source.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698