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

Unified Diff: cc/scheduler/scheduler.h

Issue 16871016: cc: Use BeginFrameArgs (Closed) Base URL: http://git.chromium.org/chromium/src.git@bfargs2
Patch Set: StateMachine test working; Reworked readback. 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 side-by-side diff with in-line comments
Download patch
Index: cc/scheduler/scheduler.h
diff --git a/cc/scheduler/scheduler.h b/cc/scheduler/scheduler.h
index 710aec43c28153ab65c400a31bf15b0c54c5060c..f980a2e168341d9dd3ff18ed236e223407eeb6a1 100644
--- a/cc/scheduler/scheduler.h
+++ b/cc/scheduler/scheduler.h
@@ -8,6 +8,7 @@
#include <string>
#include "base/basictypes.h"
+#include "base/cancelable_callback.h"
#include "base/memory/scoped_ptr.h"
#include "base/time/time.h"
#include "cc/base/cc_export.h"
@@ -20,25 +21,30 @@ namespace cc {
class Thread;
-struct ScheduledActionDrawAndSwapResult {
- ScheduledActionDrawAndSwapResult()
+struct ScheduledActionDrawSwapReadbackResult {
+ ScheduledActionDrawSwapReadbackResult()
: did_draw(false),
- did_swap(false) {}
- ScheduledActionDrawAndSwapResult(bool did_draw, bool did_swap)
+ did_swap(false),
+ did_readback(false) {}
+ ScheduledActionDrawSwapReadbackResult(bool did_draw, bool did_swap)
: did_draw(did_draw),
- did_swap(did_swap) {}
+ did_swap(did_swap),
+ did_readback(did_readback) {}
bool did_draw;
bool did_swap;
+ bool did_readback;
};
class SchedulerClient {
public:
virtual void SetNeedsBeginFrameOnImplThread(bool enable) = 0;
virtual void ScheduledActionSendBeginFrameToMainThread() = 0;
- virtual ScheduledActionDrawAndSwapResult
- ScheduledActionDrawAndSwapIfPossible() = 0;
- virtual ScheduledActionDrawAndSwapResult
- ScheduledActionDrawAndSwapForced() = 0;
+ virtual ScheduledActionDrawSwapReadbackResult
+ ScheduledActionDrawAndSwapIfPossible() = 0;
+ virtual ScheduledActionDrawSwapReadbackResult
+ ScheduledActionDrawAndSwapForced() = 0;
+ virtual ScheduledActionDrawSwapReadbackResult
+ ScheduledActionDrawAndReadback() = 0;
virtual void ScheduledActionCommit() = 0;
virtual void ScheduledActionCheckForCompletedTileUploads() = 0;
virtual void ScheduledActionActivatePendingTreeIfNeeded() = 0;
@@ -46,6 +52,9 @@ class SchedulerClient {
virtual void ScheduledActionAcquireLayerTexturesForMainThread() = 0;
virtual void DidAnticipatedDrawTimeChange(base::TimeTicks time) = 0;
virtual base::TimeDelta DrawDurationEstimate() = 0;
+ virtual void PostBeginFrameDeadline(const base::Closure& closure,
+ base::TimeTicks deadline) = 0;
+ virtual void DidBeginFrameDeadlineOnImplThread() = 0;
protected:
virtual ~SchedulerClient() {}
@@ -65,22 +74,18 @@ class CC_EXPORT Scheduler {
void SetVisible(bool visible);
void SetCanDraw(bool can_draw);
- void SetHasPendingTree(bool has_pending_tree);
+ void SetHasTrees(bool has_pending_tree, bool active_tree_is_null);
void SetNeedsCommit();
// Like SetNeedsCommit(), but ensures a commit will definitely happen even if
- // we are not visible.
- void SetNeedsForcedCommit();
+ // we are not visible. Will eventually result in a forced draw internally.
+ void SetNeedsForcedCommitForReadback();
void SetNeedsRedraw();
void SetMainThreadNeedsLayerTextures();
- // Like SetNeedsRedraw(), but ensures the draw will definitely happen even if
- // we are not visible.
- void SetNeedsForcedRedraw();
-
void DidSwapUseIncompleteTile();
void FinishCommit();
@@ -102,6 +107,7 @@ class CC_EXPORT Scheduler {
base::TimeTicks LastBeginFrameOnImplThreadTime();
void BeginFrame(const BeginFrameArgs& args);
+ void OnBeginFrameDeadline();
std::string StateAsStringForTesting() { return state_machine_.ToString(); }
@@ -109,9 +115,11 @@ class CC_EXPORT Scheduler {
Scheduler(SchedulerClient* client,
const SchedulerSettings& scheduler_settings);
+ void PostBeginFrameDeadline(base::TimeTicks deadline);
void SetupNextBeginFrameIfNeeded();
void DrawAndSwapIfPossible();
void DrawAndSwapForced();
+ void DrawAndReadback();
void ProcessScheduledActions();
const SchedulerSettings settings_;
@@ -124,6 +132,7 @@ class CC_EXPORT Scheduler {
// workaround.
bool safe_to_expect_begin_frame_;
BeginFrameArgs last_begin_frame_args_;
+ base::CancelableClosure begin_frame_deadline_closure_;
SchedulerStateMachine state_machine_;
bool inside_process_scheduled_actions_;

Powered by Google App Engine
This is Rietveld 408576698