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

Side by Side Diff: cc/scheduler/delay_based_time_source.h

Issue 18589002: cc: Fix and simplify DelayBasedTimeSource (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Use instant trace events Created 7 years, 5 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 unified diff | Download patch
« no previous file with comments | « no previous file | cc/scheduler/delay_based_time_source.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2011 The Chromium Authors. All rights reserved. 1 // Copyright 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CC_SCHEDULER_DELAY_BASED_TIME_SOURCE_H_ 5 #ifndef CC_SCHEDULER_DELAY_BASED_TIME_SOURCE_H_
6 #define CC_SCHEDULER_DELAY_BASED_TIME_SOURCE_H_ 6 #define CC_SCHEDULER_DELAY_BASED_TIME_SOURCE_H_
7 7
8 #include "base/memory/weak_ptr.h" 8 #include "base/memory/weak_ptr.h"
9 #include "cc/base/cc_export.h" 9 #include "cc/base/cc_export.h"
10 #include "cc/scheduler/time_source.h" 10 #include "cc/scheduler/time_source.h"
(...skipping 29 matching lines...) Expand all
40 40
41 protected: 41 protected:
42 DelayBasedTimeSource(base::TimeDelta interval, 42 DelayBasedTimeSource(base::TimeDelta interval,
43 base::SingleThreadTaskRunner* task_runner); 43 base::SingleThreadTaskRunner* task_runner);
44 virtual ~DelayBasedTimeSource(); 44 virtual ~DelayBasedTimeSource();
45 45
46 base::TimeTicks NextTickTarget(base::TimeTicks now); 46 base::TimeTicks NextTickTarget(base::TimeTicks now);
47 void PostNextTickTask(base::TimeTicks now); 47 void PostNextTickTask(base::TimeTicks now);
48 void OnTimerFired(); 48 void OnTimerFired();
49 49
50 enum State {
51 STATE_INACTIVE,
52 STATE_STARTING,
53 STATE_ACTIVE,
54 };
55
56 struct Parameters { 50 struct Parameters {
57 Parameters(base::TimeDelta interval, base::TimeTicks tick_target) 51 Parameters(base::TimeDelta interval, base::TimeTicks tick_target)
58 : interval(interval), tick_target(tick_target) {} 52 : interval(interval), tick_target(tick_target) {}
59 base::TimeDelta interval; 53 base::TimeDelta interval;
60 base::TimeTicks tick_target; 54 base::TimeTicks tick_target;
61 }; 55 };
62 56
63 TimeSourceClient* client_; 57 TimeSourceClient* client_;
64 bool has_tick_target_;
65 base::TimeTicks last_tick_time_; 58 base::TimeTicks last_tick_time_;
66 59
67 // current_parameters_ should only be written by PostNextTickTask. 60 // current_parameters_ should only be written by PostNextTickTask.
68 // next_parameters_ will take effect on the next call to PostNextTickTask. 61 // next_parameters_ will take effect on the next call to PostNextTickTask.
69 // Maintaining a pending set of parameters allows NextTickTime() to always 62 // Maintaining a pending set of parameters allows NextTickTime() to always
70 // reflect the actual time we expect OnTimerFired to be called. 63 // reflect the actual time we expect OnTimerFired to be called.
71 Parameters current_parameters_; 64 Parameters current_parameters_;
72 Parameters next_parameters_; 65 Parameters next_parameters_;
73 66
74 State state_; 67 bool active_;
75 68
76 base::SingleThreadTaskRunner* task_runner_; 69 base::SingleThreadTaskRunner* task_runner_;
77 base::WeakPtrFactory<DelayBasedTimeSource> weak_factory_; 70 base::WeakPtrFactory<DelayBasedTimeSource> weak_factory_;
78 71
79 private: 72 private:
80 DISALLOW_COPY_AND_ASSIGN(DelayBasedTimeSource); 73 DISALLOW_COPY_AND_ASSIGN(DelayBasedTimeSource);
81 }; 74 };
82 75
83 } // namespace cc 76 } // namespace cc
84 77
85 #endif // CC_SCHEDULER_DELAY_BASED_TIME_SOURCE_H_ 78 #endif // CC_SCHEDULER_DELAY_BASED_TIME_SOURCE_H_
OLDNEW
« no previous file with comments | « no previous file | cc/scheduler/delay_based_time_source.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698