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

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

Issue 2418593002: Fix scheduler bug in skipping main frames (Closed)
Patch Set: Rename expected draw function after danakj changes Created 4 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 <memory> 9 #include <memory>
10 #include <string> 10 #include <string>
(...skipping 170 matching lines...) Expand 10 before | Expand all | Expand 10 after
181 void DrawIfPossible(); 181 void DrawIfPossible();
182 void DrawForced(); 182 void DrawForced();
183 void ProcessScheduledActions(); 183 void ProcessScheduledActions();
184 void UpdateCompositorTimingHistoryRecordingEnabled(); 184 void UpdateCompositorTimingHistoryRecordingEnabled();
185 bool ShouldRecoverMainLatency(const BeginFrameArgs& args, 185 bool ShouldRecoverMainLatency(const BeginFrameArgs& args,
186 bool can_activate_before_deadline) const; 186 bool can_activate_before_deadline) const;
187 bool ShouldRecoverImplLatency(const BeginFrameArgs& args, 187 bool ShouldRecoverImplLatency(const BeginFrameArgs& args,
188 bool can_activate_before_deadline) const; 188 bool can_activate_before_deadline) const;
189 bool CanBeginMainFrameAndActivateBeforeDeadline( 189 bool CanBeginMainFrameAndActivateBeforeDeadline(
190 const BeginFrameArgs& args, 190 const BeginFrameArgs& args,
191 base::TimeDelta bmf_to_activate_estimate) const; 191 base::TimeDelta bmf_to_activate_estimate,
192 base::TimeTicks now) const;
192 void AdvanceCommitStateIfPossible(); 193 void AdvanceCommitStateIfPossible();
193 bool IsBeginMainFrameSentOrStarted() const; 194 bool IsBeginMainFrameSentOrStarted() const;
194 void BeginImplFrameWithDeadline(const BeginFrameArgs& args); 195 void BeginImplFrameWithDeadline(const BeginFrameArgs& args);
195 void BeginImplFrameSynchronous(const BeginFrameArgs& args); 196 void BeginImplFrameSynchronous(const BeginFrameArgs& args);
196 void BeginImplFrame(const BeginFrameArgs& args); 197 void BeginImplFrame(const BeginFrameArgs& args);
197 void FinishImplFrame(); 198 void FinishImplFrame();
198 void OnBeginImplFrameDeadline(); 199 void OnBeginImplFrameDeadline();
199 void PollToAdvanceCommitState(); 200 void PollToAdvanceCommitState();
200 201
201 bool IsInsideAction(SchedulerStateMachine::Action action) { 202 bool IsInsideAction(SchedulerStateMachine::Action action) {
202 return inside_action_ == action; 203 return inside_action_ == action;
203 } 204 }
204 205
205 base::WeakPtrFactory<Scheduler> weak_factory_; 206 base::WeakPtrFactory<Scheduler> weak_factory_;
206 207
207 DISALLOW_COPY_AND_ASSIGN(Scheduler); 208 DISALLOW_COPY_AND_ASSIGN(Scheduler);
208 }; 209 };
209 210
210 } // namespace cc 211 } // namespace cc
211 212
212 #endif // CC_SCHEDULER_SCHEDULER_H_ 213 #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