Adds fake hardware video encoder.

media/cast/sender/external_video_encoder_unittest.cc

Index: media/cast/sender/external_video_encoder_unittest.cc
diff --git a/media/cast/sender/external_video_encoder_unittest.cc b/media/cast/sender/external_video_encoder_unittest.cc
index 672771136b7b6311609e2ff1edff5585018c123a..61ac5e0b4007675681fdcb627f5fba948f362ac5 100644
--- a/media/cast/sender/external_video_encoder_unittest.cc
+++ b/media/cast/sender/external_video_encoder_unittest.cc
@@ -12,8 +12,8 @@
 #include "media/cast/cast_environment.h"
 #include "media/cast/sender/external_video_encoder.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/video_utility.h"
+#include "media/video/fake_video_encode_accelerator.h"
 #include "testing/gmock/include/gmock/gmock.h"
 
 namespace media {
@@ -105,7 +105,8 @@ class TestVideoEncoderCallback
 class ExternalVideoEncoderTest : public ::testing::Test {
  protected:
   ExternalVideoEncoderTest()
-      : test_video_encoder_callback_(new TestVideoEncoderCallback()) {
+      : stored_bitrates_(NULL),
+        test_video_encoder_callback_(new TestVideoEncoderCallback()) {
     video_config_.ssrc = 1;
     video_config_.receiver_ssrc = 2;
     video_config_.rtp_payload_type = 127;
@@ -134,8 +135,9 @@ class ExternalVideoEncoderTest : public ::testing::Test {
                             task_runner_,
                             task_runner_);
 
-    fake_vea_ = new test::FakeVideoEncodeAccelerator(task_runner_,
-                                                     &stored_bitrates_);
+    fake_vea_ = new media::FakeVideoEncodeAccelerator(task_runner_,
+                                                      std::vector<uint32>());
+    stored_bitrates_ = &fake_vea_->stored_bitrates();
     scoped_ptr<VideoEncodeAccelerator> fake_vea(fake_vea_);
     VEAFactory vea_factory(task_runner_, fake_vea.Pass());
     video_encoder_.reset(new ExternalVideoEncoder(
@@ -157,8 +159,8 @@ class ExternalVideoEncoderTest : public ::testing::Test {
   }
 
   base::SimpleTestTickClock* testing_clock_;  // Owned by CastEnvironment.
-  test::FakeVideoEncodeAccelerator* fake_vea_;  // Owned by video_encoder_.
-  std::vector<uint32> stored_bitrates_;
+  media::FakeVideoEncodeAccelerator* fake_vea_;  // Owned by video_encoder_.
+  const std::vector<uint32>* stored_bitrates_;

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

Can we replace stored_bitrates_ with fake_vea_->stored_bitrates()? Since
stored_bitrates is part of fake_vea_. There's no need to use a separate pointer
to store it.

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

Done.

   scoped_refptr<TestVideoEncoderCallback> test_video_encoder_callback_;
   VideoSenderConfig video_config_;
   scoped_refptr<test::FakeSingleThreadTaskRunner> task_runner_;
@@ -199,8 +201,8 @@ TEST_F(ExternalVideoEncoderTest, EncodePattern30fpsRunningOutOfAck) {
   video_encoder_.reset(NULL);
   task_runner_->RunTasks();
 
-  ASSERT_EQ(1u, stored_bitrates_.size());
-  EXPECT_EQ(2000u, stored_bitrates_[0]);
+  ASSERT_EQ(1u, stored_bitrates_->size());
+  EXPECT_EQ(2000u, (*stored_bitrates_)[0]);
 }
 
 TEST_F(ExternalVideoEncoderTest, StreamHeader) {
@@ -239,9 +241,9 @@ TEST(ExternalVideoEncoderEarlyDestroyTest, DestroyBeforeVEACreatedCallback) {
                           task_runner,
                           task_runner));
 
-  std::vector<uint32> stored_bitrates;
   scoped_ptr<VideoEncodeAccelerator> fake_vea(
-      new test::FakeVideoEncodeAccelerator(task_runner, &stored_bitrates));
+      new media::FakeVideoEncodeAccelerator(task_runner,
+                                            std::vector<uint32>()));
   VEAFactory vea_factory(task_runner, fake_vea.Pass());
 
   scoped_ptr<ExternalVideoEncoder> video_encoder(new ExternalVideoEncoder(