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

Unified Diff: cc/scheduler/scheduler_state_machine.h

Issue 1265023005: cc: Add SchedulerStateMachine::DidDraw and use for forced draws (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@WillDidAction0
Patch Set: Created 5 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « cc/scheduler/scheduler.cc ('k') | cc/scheduler/scheduler_state_machine.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/scheduler/scheduler_state_machine.h
diff --git a/cc/scheduler/scheduler_state_machine.h b/cc/scheduler/scheduler_state_machine.h
index 21d153cf2b5433be82890067f35ce3d1ce3a82da..0aedb036d97697c73d2de54ba2647f36a1430abc 100644
--- a/cc/scheduler/scheduler_state_machine.h
+++ b/cc/scheduler/scheduler_state_machine.h
@@ -123,6 +123,7 @@ class CC_EXPORT SchedulerStateMachine {
Action NextAction() const;
void WillAction(Action action);
+ void DidAction(Action action);
sunnyps 2015/08/25 17:23:24 nit: DidPerformAction?
// Indicates whether the impl thread needs a BeginImplFrame callback in order
// to make progress.
@@ -191,8 +192,8 @@ class CC_EXPORT SchedulerStateMachine {
return impl_latency_takes_priority_;
}
- // Indicates whether ACTION_DRAW_AND_SWAP_IF_POSSIBLE drew to the screen.
- void DidDrawIfPossibleCompleted(DrawResult result);
+ // Indicates whether a draw request succeeded or not.
+ void SetDrawResult(DrawResult result);
// Indicates that a new begin main frame flow needs to be performed, either
// to pull updates from the main thread to the impl, or to push deltas from
@@ -288,11 +289,13 @@ class CC_EXPORT SchedulerStateMachine {
void WillSendBeginMainFrame();
void WillCommit(bool commit_had_no_updates);
void WillActivate();
- void WillDraw(bool did_request_swap);
+ void WillDraw();
void WillBeginOutputSurfaceCreation();
void WillPrepareTiles();
void WillInvalidateOutputSurface();
+ void DidDraw(bool did_request_swap);
+
const SchedulerSettings settings_;
OutputSurfaceState output_surface_state_;
@@ -326,6 +329,7 @@ class CC_EXPORT SchedulerStateMachine {
int pending_swaps_;
int swaps_with_current_output_surface_;
bool needs_redraw_;
+ DrawResult last_draw_result_;
bool needs_animate_;
bool needs_prepare_tiles_;
bool needs_begin_main_frame_;
« no previous file with comments | « cc/scheduler/scheduler.cc ('k') | cc/scheduler/scheduler_state_machine.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698