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

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

Issue 554973002: Disable scheduler deadline task on battery power in Windows (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address brianderson's and mithro's comments Created 6 years, 2 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/scheduler.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_SCHEDULER_H_ 5 #ifndef CC_SCHEDULER_SCHEDULER_H_
6 #define CC_SCHEDULER_SCHEDULER_H_ 6 #define CC_SCHEDULER_SCHEDULER_H_
7 7
8 #include <deque> 8 #include <deque>
9 #include <string> 9 #include <string>
10 10
11 #include "base/basictypes.h" 11 #include "base/basictypes.h"
12 #include "base/cancelable_callback.h" 12 #include "base/cancelable_callback.h"
13 #include "base/memory/scoped_ptr.h" 13 #include "base/memory/scoped_ptr.h"
14 #include "base/power_monitor/power_monitor.h"
15 #include "base/power_monitor/power_observer.h"
14 #include "base/time/time.h" 16 #include "base/time/time.h"
15 #include "cc/base/cc_export.h" 17 #include "cc/base/cc_export.h"
16 #include "cc/output/begin_frame_args.h" 18 #include "cc/output/begin_frame_args.h"
17 #include "cc/output/vsync_parameter_observer.h" 19 #include "cc/output/vsync_parameter_observer.h"
18 #include "cc/scheduler/begin_frame_source.h" 20 #include "cc/scheduler/begin_frame_source.h"
19 #include "cc/scheduler/delay_based_time_source.h" 21 #include "cc/scheduler/delay_based_time_source.h"
20 #include "cc/scheduler/draw_result.h" 22 #include "cc/scheduler/draw_result.h"
21 #include "cc/scheduler/scheduler_settings.h" 23 #include "cc/scheduler/scheduler_settings.h"
22 #include "cc/scheduler/scheduler_state_machine.h" 24 #include "cc/scheduler/scheduler_state_machine.h"
23 25
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after
64 virtual BeginFrameSource* ConstructPrimaryFrameSource(Scheduler* scheduler); 66 virtual BeginFrameSource* ConstructPrimaryFrameSource(Scheduler* scheduler);
65 virtual BeginFrameSource* ConstructBackgroundFrameSource( 67 virtual BeginFrameSource* ConstructBackgroundFrameSource(
66 Scheduler* scheduler); 68 Scheduler* scheduler);
67 69
68 protected: 70 protected:
69 SchedulerFrameSourcesConstructor() {} 71 SchedulerFrameSourcesConstructor() {}
70 72
71 friend class Scheduler; 73 friend class Scheduler;
72 }; 74 };
73 75
74 class CC_EXPORT Scheduler : public BeginFrameObserverMixIn { 76 class CC_EXPORT Scheduler : public BeginFrameObserverMixIn,
77 public base::PowerObserver {
75 public: 78 public:
76 static scoped_ptr<Scheduler> Create( 79 static scoped_ptr<Scheduler> Create(
77 SchedulerClient* client, 80 SchedulerClient* client,
78 const SchedulerSettings& scheduler_settings, 81 const SchedulerSettings& scheduler_settings,
79 int layer_tree_host_id, 82 int layer_tree_host_id,
80 const scoped_refptr<base::SingleThreadTaskRunner>& task_runner) { 83 const scoped_refptr<base::SingleThreadTaskRunner>& task_runner) {
81 SchedulerFrameSourcesConstructor frame_sources_constructor; 84 SchedulerFrameSourcesConstructor frame_sources_constructor;
82 return make_scoped_ptr(new Scheduler(client, 85 return make_scoped_ptr(new Scheduler(client,
83 scheduler_settings, 86 scheduler_settings,
84 layer_tree_host_id, 87 layer_tree_host_id,
85 task_runner, 88 task_runner,
86 &frame_sources_constructor)); 89 &frame_sources_constructor,
90 base::PowerMonitor::Get()));
danakj 2014/10/15 15:42:26 I think brian meant that you'd pass it to Create()
87 } 91 }
88 92
89 virtual ~Scheduler(); 93 virtual ~Scheduler();
90 94
95 // base::PowerObserver method.
96 virtual void OnPowerStateChange(bool on_battery_power) override;
97
91 const SchedulerSettings& settings() const { return settings_; } 98 const SchedulerSettings& settings() const { return settings_; }
92 99
93 void CommitVSyncParameters(base::TimeTicks timebase, 100 void CommitVSyncParameters(base::TimeTicks timebase,
94 base::TimeDelta interval); 101 base::TimeDelta interval);
95 void SetEstimatedParentDrawTime(base::TimeDelta draw_time); 102 void SetEstimatedParentDrawTime(base::TimeDelta draw_time);
96 103
97 void SetCanStart(); 104 void SetCanStart();
98 105
99 void SetVisible(bool visible); 106 void SetVisible(bool visible);
100 void SetCanDraw(bool can_draw); 107 void SetCanDraw(bool can_draw);
(...skipping 56 matching lines...) Expand 10 before | Expand all | Expand 10 after
157 } 164 }
158 165
159 // BeginFrameObserverMixin 166 // BeginFrameObserverMixin
160 virtual bool OnBeginFrameMixInDelegate(const BeginFrameArgs& args) override; 167 virtual bool OnBeginFrameMixInDelegate(const BeginFrameArgs& args) override;
161 168
162 protected: 169 protected:
163 Scheduler(SchedulerClient* client, 170 Scheduler(SchedulerClient* client,
164 const SchedulerSettings& scheduler_settings, 171 const SchedulerSettings& scheduler_settings,
165 int layer_tree_host_id, 172 int layer_tree_host_id,
166 const scoped_refptr<base::SingleThreadTaskRunner>& task_runner, 173 const scoped_refptr<base::SingleThreadTaskRunner>& task_runner,
167 SchedulerFrameSourcesConstructor* frame_sources_constructor); 174 SchedulerFrameSourcesConstructor* frame_sources_constructor,
175 base::PowerMonitor* power_monitor);
168 176
169 // virtual for testing - Don't call these in the constructor or 177 // virtual for testing - Don't call these in the constructor or
170 // destructor! 178 // destructor!
171 virtual base::TimeTicks Now() const; 179 virtual base::TimeTicks Now() const;
172 180
173 scoped_ptr<BeginFrameSourceMultiplexer> frame_source_; 181 scoped_ptr<BeginFrameSourceMultiplexer> frame_source_;
174 BeginFrameSource* primary_frame_source_; 182 BeginFrameSource* primary_frame_source_;
175 BeginFrameSource* background_frame_source_; 183 BeginFrameSource* background_frame_source_;
176 184
177 // Storage when frame sources are internal 185 // Storage when frame sources are internal
178 scoped_ptr<BeginFrameSource> primary_frame_source_internal_; 186 scoped_ptr<BeginFrameSource> primary_frame_source_internal_;
179 scoped_ptr<SyntheticBeginFrameSource> background_frame_source_internal_; 187 scoped_ptr<SyntheticBeginFrameSource> background_frame_source_internal_;
180 188
181 VSyncParameterObserver* vsync_observer_; 189 VSyncParameterObserver* vsync_observer_;
182 190
183 const SchedulerSettings settings_; 191 const SchedulerSettings settings_;
184 SchedulerClient* client_; 192 SchedulerClient* client_;
185 int layer_tree_host_id_; 193 int layer_tree_host_id_;
186 scoped_refptr<base::SingleThreadTaskRunner> task_runner_; 194 scoped_refptr<base::SingleThreadTaskRunner> task_runner_;
187 195
196 base::PowerMonitor* power_monitor_;
197
188 base::TimeDelta estimated_parent_draw_time_; 198 base::TimeDelta estimated_parent_draw_time_;
189 199
190 bool begin_retro_frame_posted_; 200 bool begin_retro_frame_posted_;
191 std::deque<BeginFrameArgs> begin_retro_frame_args_; 201 std::deque<BeginFrameArgs> begin_retro_frame_args_;
192 BeginFrameArgs begin_impl_frame_args_; 202 BeginFrameArgs begin_impl_frame_args_;
193 203
194 base::Closure begin_retro_frame_closure_; 204 base::Closure begin_retro_frame_closure_;
195 base::Closure begin_unthrottled_frame_closure_; 205 base::Closure begin_unthrottled_frame_closure_;
196 206
197 base::Closure begin_impl_frame_deadline_closure_; 207 base::Closure begin_impl_frame_deadline_closure_;
(...skipping 18 matching lines...) Expand all
216 void DrawAndSwapIfPossible(); 226 void DrawAndSwapIfPossible();
217 void ProcessScheduledActions(); 227 void ProcessScheduledActions();
218 bool CanCommitAndActivateBeforeDeadline() const; 228 bool CanCommitAndActivateBeforeDeadline() const;
219 void AdvanceCommitStateIfPossible(); 229 void AdvanceCommitStateIfPossible();
220 bool IsBeginMainFrameSentOrStarted() const; 230 bool IsBeginMainFrameSentOrStarted() const;
221 void BeginRetroFrame(); 231 void BeginRetroFrame();
222 void BeginImplFrame(const BeginFrameArgs& args); 232 void BeginImplFrame(const BeginFrameArgs& args);
223 void OnBeginImplFrameDeadline(); 233 void OnBeginImplFrameDeadline();
224 void PollForAnticipatedDrawTriggers(); 234 void PollForAnticipatedDrawTriggers();
225 void PollToAdvanceCommitState(); 235 void PollToAdvanceCommitState();
236 void SetupPowerMonitoring();
237 void TeardownPowerMonitoring();
226 238
227 base::TimeDelta EstimatedParentDrawTime() { 239 base::TimeDelta EstimatedParentDrawTime() {
228 return estimated_parent_draw_time_; 240 return estimated_parent_draw_time_;
229 } 241 }
230 242
231 bool IsInsideAction(SchedulerStateMachine::Action action) { 243 bool IsInsideAction(SchedulerStateMachine::Action action) {
232 return inside_action_ == action; 244 return inside_action_ == action;
233 } 245 }
234 246
235 base::WeakPtrFactory<Scheduler> weak_factory_; 247 base::WeakPtrFactory<Scheduler> weak_factory_;
236 248
237 friend class SchedulerFrameSourcesConstructor; 249 friend class SchedulerFrameSourcesConstructor;
238 friend class TestSchedulerFrameSourcesConstructor; 250 friend class TestSchedulerFrameSourcesConstructor;
239 251
240 DISALLOW_COPY_AND_ASSIGN(Scheduler); 252 DISALLOW_COPY_AND_ASSIGN(Scheduler);
241 }; 253 };
242 254
243 } // namespace cc 255 } // namespace cc
244 256
245 #endif // CC_SCHEDULER_SCHEDULER_H_ 257 #endif // CC_SCHEDULER_SCHEDULER_H_
OLDNEW
« no previous file with comments | « no previous file | cc/scheduler/scheduler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698