cc: Remove BeginFrameSourcesConstructor.

cc/surfaces/display_scheduler_unittest.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "cc/surfaces/display_scheduler.h"
 
 #include "base/logging.h"
 #include "base/test/null_task_runner.h"
 #include "base/test/simple_test_tick_clock.h"
 #include "base/trace_event/trace_event.h"
@@ skipping 21 matching lines @@
   int draw_and_swap_count() const { return draw_and_swap_count_; }
 
  protected:
   int draw_and_swap_count_;
 };
 
 class TestDisplayScheduler : public DisplayScheduler {
  public:
   TestDisplayScheduler(DisplaySchedulerClient* client,
                        BeginFrameSource* begin_frame_source,
-                       scoped_refptr<base::SingleThreadTaskRunner> task_runner,
+                       base::NullTaskRunner* task_runner,

[mithro-old, 2015/06/22 11:49:39]

Why is this changing to a NullTaskRunner verses a SingleThreadTaskRunner?

[sunnyps, 2015/06/22 20:32:41]

No particular reason but I like keeping tests specific as far as possible.

                        int max_pending_swaps)
       : DisplayScheduler(client,
                          begin_frame_source,
                          task_runner,
                          max_pending_swaps),
         scheduler_begin_frame_deadline_count_(0) {}
 
   base::TimeTicks DesiredBeginFrameDeadlineTimeForTest() {
     return DesiredBeginFrameDeadlineTime();
   }
@@ skipping 15 matching lines @@
 
 class DisplaySchedulerTest : public testing::Test {
  public:
   DisplaySchedulerTest() : now_src_(new base::SimpleTestTickClock()) {
     const int max_pending_swaps = 1;
     now_src_->Advance(base::TimeDelta::FromMicroseconds(10000));
     null_task_runner_ = make_scoped_refptr(new base::NullTaskRunner);
     client_ = make_scoped_ptr(new FakeDisplaySchedulerClient);
     scheduler_ = make_scoped_ptr(
         new TestDisplayScheduler(client_.get(), &fake_begin_frame_source_,
-                                 null_task_runner_, max_pending_swaps));
+                                 null_task_runner_.get(), max_pending_swaps));
   }
 
   ~DisplaySchedulerTest() override {}
 
   void SetUp() override { scheduler_->SetRootSurfaceResourcesLocked(false); }
 
   void BeginFrameForTest() {
     base::TimeTicks frame_time = now_src_->NowTicks();
     base::TimeDelta interval = BeginFrameArgs::DefaultInterval();
     base::TimeTicks deadline = frame_time + interval;
@@ skipping 246 matching lines @@
 
   scheduler_->SetRootSurfaceResourcesLocked(true);
   EXPECT_EQ(count++, scheduler_->scheduler_begin_frame_deadline_count());
 
   scheduler_->OutputSurfaceLost();
   EXPECT_EQ(count++, scheduler_->scheduler_begin_frame_deadline_count());
 }
 
 }  // namespace
 }  // namespace cc