Adds fake hardware video encoder.

media/cast/sender/video_sender_unittest.cc

Index: media/cast/sender/video_sender_unittest.cc
diff --git a/media/cast/sender/video_sender_unittest.cc b/media/cast/sender/video_sender_unittest.cc
index 917e4a9c2636c15ec844ebcfe96e100b8e59b5a1..b8659259c061b550dce0fb2d134ea97f99266a21 100644
--- a/media/cast/sender/video_sender_unittest.cc
+++ b/media/cast/sender/video_sender_unittest.cc
@@ -18,9 +18,9 @@
 #include "media/cast/sender/video_frame_factory.h"
 #include "media/cast/sender/video_sender.h"
 #include "media/cast/test/fake_single_thread_task_runner.h"
-#include "media/cast/test/fake_video_encode_accelerator.h"
 #include "media/cast/test/utility/default_config.h"
 #include "media/cast/test/utility/video_utility.h"
+#include "media/video/fake_video_encode_accelerator.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
@@ -133,7 +133,8 @@ class PeerVideoSender : public VideoSender {
 
 class VideoSenderTest : public ::testing::Test {
  protected:
-  VideoSenderTest() {
+  VideoSenderTest()
+      : stored_bitrates_(NULL) {
     testing_clock_ = new base::SimpleTestTickClock();
     testing_clock_->Advance(base::TimeTicks::Now() - base::TimeTicks());
     task_runner_ = new test::FakeSingleThreadTaskRunner(testing_clock_);
@@ -189,9 +190,10 @@ class VideoSenderTest : public ::testing::Test {
     CastInitializationStatus status = STATUS_VIDEO_UNINITIALIZED;
 
     if (external) {
-      test::FakeVideoEncodeAccelerator* fake_vea =
-          new test::FakeVideoEncodeAccelerator(
-              task_runner_, &stored_bitrates_);
+      media::FakeVideoEncodeAccelerator* fake_vea =
+          new media::FakeVideoEncodeAccelerator(
+              task_runner_, std::vector<uint32>());
+      stored_bitrates_ = &fake_vea->stored_bitrates();
       fake_vea->SetWillInitializationSucceed(expect_init_success);
       scoped_ptr<VideoEncodeAccelerator> fake_vea_owner(fake_vea);
       video_sender_.reset(
@@ -251,7 +253,7 @@ class VideoSenderTest : public ::testing::Test {
   scoped_ptr<CastTransportSenderImpl> transport_sender_;
   scoped_refptr<test::FakeSingleThreadTaskRunner> task_runner_;
   scoped_ptr<PeerVideoSender> video_sender_;
-  std::vector<uint32> stored_bitrates_;
+  const std::vector<uint32>* stored_bitrates_;

[wuchengli, 2014/12/09 14:29:25]

This is not very good because it is a part of PeerVideoSender. Also, I think
it's better to use googlemock EXPECT_CALL to count the number of method call. We
can have a mock class to inherit from FakeVEA. But there's no need to fix it in
this CL.

[hellner1, 2014/12/10 22:38:59]

Agree. No action then as changing this would bloat the cl.

   scoped_refptr<CastEnvironment> cast_environment_;
   int last_pixel_value_;
   base::TimeTicks first_frame_timestamp_;
@@ -286,7 +288,7 @@ TEST_F(VideoSenderTest, ExternalEncoder) {
 
   // Fixed bitrate is used for external encoder. Bitrate is only once
   // to the encoder.
-  EXPECT_EQ(1u, stored_bitrates_.size());
+  EXPECT_EQ(1u, stored_bitrates_->size());
   video_sender_.reset(NULL);
   task_runner_->RunTasks();
 }