Refactor VideoRendererImpl to use VideoFrameScheduler.

media/filters/video_frame_scheduler_impl_unittest.cc

 // Copyright 2014 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 "base/message_loop/message_loop.h"
 #include "base/test/simple_test_tick_clock.h"
 #include "media/base/test_helpers.h"
 #include "media/base/video_frame.h"
 #include "media/filters/video_frame_scheduler_impl.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace media {
 
+using testing::_;
+
 // NOTE: millisecond-level resolution is used for times as real delayed tasks
 // are posted. Don't use large values if you want to keep tests running fast.
 class VideoFrameSchedulerImplTest : public testing::Test {
  public:
   VideoFrameSchedulerImplTest()
       : scheduler_(message_loop_.message_loop_proxy(),
                    base::Bind(&VideoFrameSchedulerImplTest::OnDisplay,
                               base::Unretained(this))),
         tick_clock_(new base::SimpleTestTickClock()) {
     scheduler_.SetTickClockForTesting(scoped_ptr<base::TickClock>(tick_clock_));
@@ skipping 106 matching lines @@
   EXPECT_CALL(*this, OnFrameDone(displayed, VideoFrameScheduler::DISPLAYED));
   AdvanceTime(30);
   RunUntilTimeHasElapsed(30);
 }
 
 TEST_F(VideoFrameSchedulerImplTest, Reset) {
   scoped_refptr<VideoFrame> frame =
       VideoFrame::CreateBlackFrame(gfx::Size(8, 8));
   Schedule(frame, 10);
 
-  // Despite being on time, frames are returned immediately.
-  EXPECT_CALL(*this, OnFrameDone(frame, VideoFrameScheduler::RESET));
+  // Despite being on time, frame callback isn't run.
+  EXPECT_CALL(*this, OnFrameDone(_, _)).Times(0);
   AdvanceTime(10);
   Reset();
   RunUntilTimeHasElapsed(10);
 }
 
 }  // namespace media