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

Unified Diff: cc/surfaces/display_scheduler_unittest.cc

Issue 2583483002: [cc] Adds source_id and sequence_number to BeginFrameArgs. (Closed)
Patch Set: Address Sami's comments Created 4 years 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/surfaces/display_scheduler_unittest.cc
diff --git a/cc/surfaces/display_scheduler_unittest.cc b/cc/surfaces/display_scheduler_unittest.cc
index ec9c2214405bd8eebab4fd9c9d482bc07d92ccd1..03fe85c2241ca98f0ebbea893d2058e304f0b351 100644
--- a/cc/surfaces/display_scheduler_unittest.cc
+++ b/cc/surfaces/display_scheduler_unittest.cc
@@ -73,6 +73,7 @@ class DisplaySchedulerTest : public testing::Test {
public:
DisplaySchedulerTest()
: fake_begin_frame_source_(0.f, false),
+ next_begin_frame_number_(BeginFrameArgs::kStartingFrameNumber),
task_runner_(new base::NullTaskRunner),
scheduler_(&fake_begin_frame_source_,
task_runner_.get(),
@@ -89,9 +90,10 @@ class DisplaySchedulerTest : public testing::Test {
base::TimeTicks frame_time = now_src_.NowTicks();
base::TimeDelta interval = BeginFrameArgs::DefaultInterval();
base::TimeTicks deadline = frame_time + interval;
- fake_begin_frame_source_.TestOnBeginFrame(
- BeginFrameArgs::Create(BEGINFRAME_FROM_HERE, frame_time, deadline,
- interval, BeginFrameArgs::NORMAL));
+ fake_begin_frame_source_.TestOnBeginFrame(BeginFrameArgs::Create(
+ BEGINFRAME_FROM_HERE, fake_begin_frame_source_.source_id(),
+ next_begin_frame_number_++, frame_time, deadline, interval,
+ BeginFrameArgs::NORMAL));
}
protected:
@@ -100,6 +102,7 @@ class DisplaySchedulerTest : public testing::Test {
DisplayScheduler& scheduler() { return scheduler_; }
FakeExternalBeginFrameSource fake_begin_frame_source_;
+ uint64_t next_begin_frame_number_;
base::SimpleTestTickClock now_src_;
scoped_refptr<base::NullTaskRunner> task_runner_;

Powered by Google App Engine
This is Rietveld 408576698