[cc] Adds source_id and sequence_number to BeginFrameArgs.

cc/surfaces/display_scheduler_unittest.cc

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_;

[brianderson, 2016/12/15 22:10:33]

In this case, does it make sense for this to be tracked by the
FakeExternalBeginFrameSource?

[Eric Seckler, 2016/12/16 17:25:58]

true, that's nicer :) Done.

 
   base::SimpleTestTickClock now_src_;
   scoped_refptr<base::NullTaskRunner> task_runner_;