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

Unified Diff: cc/scheduler/scheduler_state_machine.h

Issue 16871016: cc: Use BeginFrameArgs (Closed) Base URL: http://git.chromium.org/chromium/src.git@bfargs2
Patch Set: Tweaks; Address more comments; Created 7 years, 6 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
Index: cc/scheduler/scheduler_state_machine.h
diff --git a/cc/scheduler/scheduler_state_machine.h b/cc/scheduler/scheduler_state_machine.h
index 3784f6f3763391c656516ee448c2615e5ea6a474..597ff51adb40e66d47b0bcf17dd7edec845fed45 100644
--- a/cc/scheduler/scheduler_state_machine.h
+++ b/cc/scheduler/scheduler_state_machine.h
@@ -31,10 +31,22 @@ class CC_EXPORT SchedulerStateMachine {
// settings must be valid for the lifetime of this class.
explicit SchedulerStateMachine(const SchedulerSettings& settings);
+ // Note: BeginFrameState will always cycle through all the states in order.
+ // Whether or not it actually waits or draws, it will at least try to wait in
+ // BEGIN_FRAME_STATE_DEADLINE_PENDING and try to draw in
+ // BEGIN_FRAME_STATE_INSIDE_DEADLINE
+ enum BeginFrameState {
+ BEGIN_FRAME_STATE_IDLE,
+ BEGIN_FRAME_STATE_INSIDE_BEGIN_FRAME,
+ BEGIN_FRAME_STATE_DEADLINE_PENDING,
+ BEGIN_FRAME_STATE_INSIDE_DEADLINE,
+ };
+
enum CommitState {
COMMIT_STATE_IDLE,
COMMIT_STATE_FRAME_IN_PROGRESS,
COMMIT_STATE_READY_TO_COMMIT,
+ COMMIT_STATE_WAITING_FOR_ACTIVATION,
COMMIT_STATE_WAITING_FOR_FIRST_DRAW,
COMMIT_STATE_WAITING_FOR_FIRST_FORCED_DRAW,
};
@@ -80,9 +92,10 @@ class CC_EXPORT SchedulerStateMachine {
// Indicates that the system has entered and left a BeginFrame callback.
// The scheduler will not draw more than once in a given BeginFrame
// callback.
- void DidEnterBeginFrame(const BeginFrameArgs& args);
- void DidLeaveBeginFrame();
- bool inside_begin_frame() const { return inside_begin_frame_; }
+ void OnBeginFrame(const BeginFrameArgs& args);
+ void OnBeginFrameDeadline();
+ bool ShouldTriggerBeginFrameDeadlineEarly() const;
+ bool InsideBeginFrame() const;
// Indicates whether the LayerTreeHostImpl is visible.
void SetVisible(bool visible);
@@ -161,23 +174,29 @@ class CC_EXPORT SchedulerStateMachine {
protected:
bool ShouldDrawForced() const;
bool ScheduledToDraw() const;
+ bool WillDrawNewTree() const;
bool ShouldDraw() const;
+ bool ShouldDrawInternal(BeginFrameState begin_frame_state) const;
bool ShouldAttemptTreeActivation() const;
bool ShouldAcquireLayerTexturesForMainThread() const;
bool ShouldCheckForCompletedTileUploads() const;
+ bool ShouldSendBeginFrameToMainThread() const;
bool HasDrawnThisFrame() const;
- bool HasAttemptedTreeActivationThisFrame() const;
- bool HasCheckedForCompletedTileUploadsThisFrame() const;
+ bool HasAttemptedTreeActivationThisDrawAttempt() const;
+ bool HasCheckedForCompletedTileUploadsThisDrawAttempt() const;
+ bool HasSentBeginFrameToMainThreadThisFrame() const;
const SchedulerSettings settings_;
CommitState commit_state_;
int commit_count_;
- int current_frame_number_;
- int last_frame_number_where_draw_was_called_;
- int last_frame_number_where_tree_activation_attempted_;
- int last_frame_number_where_check_for_completed_tile_uploads_called_;
+ int begin_frame_count_;
+ int draw_attempt_count_;
+ int last_begin_frame_count_draw_was_called_;
+ int last_begin_frame_count_begin_frame_sent_to_main_thread_;
+ int last_draw_attempt_count_tree_activation_attempted_;
+ int last_draw_attempt_count_completed_tile_uploads_checked_;
int consecutive_failed_draws_;
int maximum_number_of_failed_draws_before_draw_is_forced_;
bool needs_redraw_;
@@ -188,7 +207,9 @@ class CC_EXPORT SchedulerStateMachine {
bool needs_forced_commit_;
bool expect_immediate_begin_frame_for_main_thread_;
bool main_thread_needs_layer_textures_;
- bool inside_begin_frame_;
+ BeginFrameState begin_frame_state_;
+ bool commit_tree_has_been_drawn_or_null_;
+ bool active_tree_has_been_drawn_or_null_;
BeginFrameArgs last_begin_frame_args_;
bool visible_;
bool can_start_;

Powered by Google App Engine
This is Rietveld 408576698