Revert r6110 and r6109.

Effectively re-landing r6104 as well as r6108 which includes a fix to a
compile error that caused r6104 to be reverted in r6110.

BUG=
TBR=henrike@webrtc.org

Review URL: https://webrtc-codereview.appspot.com/20459004

git-svn-id: http://webrtc.googlecode.com/svn/trunk/talk@6119 4adac7df-926f-26a2-2b94-8c16560cd09d
diff --git a/media/webrtc/webrtcvideoengine2_unittest.cc b/media/webrtc/webrtcvideoengine2_unittest.cc
new file mode 100644
index 0000000..d21899d
--- /dev/null
+++ b/media/webrtc/webrtcvideoengine2_unittest.cc
@@ -0,0 +1,1187 @@
+/*
+ * libjingle
+ * Copyright 2004 Google Inc.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions are met:
+ *
+ *  1. Redistributions of source code must retain the above copyright notice,
+ *     this list of conditions and the following disclaimer.
+ *  2. Redistributions in binary form must reproduce the above copyright notice,
+ *     this list of conditions and the following disclaimer in the documentation
+ *     and/or other materials provided with the distribution.
+ *  3. The name of the author may not be used to endorse or promote products
+ *     derived from this software without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE AUTHOR ``AS IS'' AND ANY EXPRESS OR IMPLIED
+ * WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF
+ * MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO
+ * EVENT SHALL THE AUTHOR BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+ * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
+ * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS;
+ * OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY,
+ * WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR
+ * OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF
+ * ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#include <map>
+
+#include "talk/base/gunit.h"
+#include "talk/media/base/testutils.h"
+#include "talk/media/base/videoengine_unittest.h"
+#include "talk/media/webrtc/webrtcvideoengine2.h"
+#include "talk/media/webrtc/webrtcvideochannelfactory.h"
+#include "webrtc/call.h"
+
+namespace {
+static const cricket::VideoCodec kVp8Codec720p(100, "VP8", 1280, 720, 30, 0);
+static const cricket::VideoCodec kVp8Codec360p(100, "VP8", 640, 360, 30, 0);
+static const cricket::VideoCodec kVp8Codec270p(100, "VP8", 480, 270, 30, 0);
+static const cricket::VideoCodec kVp8Codec180p(100, "VP8", 320, 180, 30, 0);
+
+static const cricket::VideoCodec kVp8Codec(100, "VP8", 640, 400, 30, 0);
+static const cricket::VideoCodec kVp9Codec(101, "VP9", 640, 400, 30, 0);
+static const cricket::VideoCodec kRedCodec(116, "red", 0, 0, 0, 0);
+static const cricket::VideoCodec kUlpfecCodec(117, "ulpfec", 0, 0, 0, 0);
+
+static const uint32 kSsrcs1[] = {1};
+static const uint32 kRtxSsrcs1[] = {4};
+}  // namespace
+
+namespace cricket {
+class FakeVideoSendStream : public webrtc::VideoSendStream {
+ public:
+  explicit FakeVideoSendStream(const webrtc::VideoSendStream::Config& config)
+      : sending_(false) {
+    config_ = config;
+  }
+
+  webrtc::VideoSendStream::Config GetConfig() { return config_; }
+
+  bool IsSending() { return sending_; }
+
+ private:
+  virtual webrtc::VideoSendStream::Stats GetStats() const OVERRIDE {
+    return webrtc::VideoSendStream::Stats();
+  }
+
+  virtual bool ReconfigureVideoEncoder(
+      const std::vector<webrtc::VideoStream>& streams,
+      void* encoder_specific) OVERRIDE {
+    // TODO(pbos): Store encoder_specific ptr?
+    config_.encoder_settings.streams = streams;
+    return true;
+  }
+
+  virtual webrtc::VideoSendStreamInput* Input() OVERRIDE {
+    // TODO(pbos): Fix.
+    return NULL;
+  }
+
+  virtual void Start() OVERRIDE { sending_ = true; }
+
+  virtual void Stop() OVERRIDE { sending_ = false; }
+
+  bool sending_;
+  webrtc::VideoSendStream::Config config_;
+};
+
+class FakeVideoReceiveStream : public webrtc::VideoReceiveStream {
+ public:
+  explicit FakeVideoReceiveStream(
+      const webrtc::VideoReceiveStream::Config& config)
+      : config_(config), receiving_(false) {}
+
+  webrtc::VideoReceiveStream::Config GetConfig() { return config_; }
+
+ private:
+  virtual webrtc::VideoReceiveStream::Stats GetStats() const OVERRIDE {
+    return webrtc::VideoReceiveStream::Stats();
+  }
+
+  virtual void Start() OVERRIDE { receiving_ = true; }
+  virtual void Stop() OVERRIDE { receiving_ = false; }
+  virtual void GetCurrentReceiveCodec(webrtc::VideoCodec* codec) OVERRIDE {}
+
+  webrtc::VideoReceiveStream::Config config_;
+  bool receiving_;
+};
+
+class FakeCall : public webrtc::Call {
+ public:
+  FakeCall() { SetVideoCodecs(GetDefaultVideoCodecs()); }
+
+  ~FakeCall() {
+    EXPECT_EQ(0u, video_send_streams_.size());
+    EXPECT_EQ(0u, video_receive_streams_.size());
+  }
+
+  void SetVideoCodecs(const std::vector<webrtc::VideoCodec> codecs) {
+    codecs_ = codecs;
+  }
+
+  std::vector<FakeVideoSendStream*> GetVideoSendStreams() {
+    return video_send_streams_;
+  }
+
+  std::vector<FakeVideoReceiveStream*> GetVideoReceiveStreams() {
+    return video_receive_streams_;
+  }
+
+  webrtc::VideoCodec GetEmptyVideoCodec() {
+    webrtc::VideoCodec codec;
+    codec.minBitrate = 300;
+    codec.startBitrate = 800;
+    codec.maxBitrate = 1500;
+    codec.maxFramerate = 10;
+    codec.width = 640;
+    codec.height = 480;
+    codec.qpMax = 56;
+
+    return codec;
+  }
+
+  webrtc::VideoCodec GetVideoCodecVp8() {
+    webrtc::VideoCodec vp8_codec = GetEmptyVideoCodec();
+    vp8_codec.codecType = webrtc::kVideoCodecVP8;
+    strcpy(vp8_codec.plName, kVp8Codec.name.c_str());
+    vp8_codec.plType = kVp8Codec.id;
+
+    return vp8_codec;
+  }
+
+  webrtc::VideoCodec GetVideoCodecVp9() {
+    webrtc::VideoCodec vp9_codec = GetEmptyVideoCodec();
+    // TODO(pbos): Add a correct codecType when webrtc has one.
+    vp9_codec.codecType = webrtc::kVideoCodecVP8;
+    strcpy(vp9_codec.plName, kVp9Codec.name.c_str());
+    vp9_codec.plType = kVp9Codec.id;
+
+    return vp9_codec;
+  }
+
+  std::vector<webrtc::VideoCodec> GetDefaultVideoCodecs() {
+    std::vector<webrtc::VideoCodec> codecs;
+    codecs.push_back(GetVideoCodecVp8());
+    //    codecs.push_back(GetVideoCodecVp9());
+
+    return codecs;
+  }
+
+ private:
+  virtual webrtc::VideoSendStream::Config GetDefaultSendConfig() OVERRIDE {
+    webrtc::VideoSendStream::Config config;
+    // TODO(pbos): Encoder settings.
+    //    config.codec = GetVideoCodecVp8();
+    return config;
+  }
+
+  virtual webrtc::VideoSendStream* CreateVideoSendStream(
+      const webrtc::VideoSendStream::Config& config) OVERRIDE {
+    FakeVideoSendStream* fake_stream = new FakeVideoSendStream(config);
+    video_send_streams_.push_back(fake_stream);
+    return fake_stream;
+  }
+
+  virtual void DestroyVideoSendStream(webrtc::VideoSendStream* send_stream)
+      OVERRIDE {
+    FakeVideoSendStream* fake_stream =
+        static_cast<FakeVideoSendStream*>(send_stream);
+    for (size_t i = 0; i < video_send_streams_.size(); ++i) {
+      if (video_send_streams_[i] == fake_stream) {
+        delete video_send_streams_[i];
+        video_send_streams_.erase(video_send_streams_.begin() + i);
+        return;
+      }
+    }
+    ADD_FAILURE() << "DestroyVideoSendStream called with unknown paramter.";
+  }
+
+  virtual webrtc::VideoReceiveStream::Config GetDefaultReceiveConfig()
+      OVERRIDE {
+    return webrtc::VideoReceiveStream::Config();
+  }
+
+  virtual webrtc::VideoReceiveStream* CreateVideoReceiveStream(
+      const webrtc::VideoReceiveStream::Config& config) OVERRIDE {
+    video_receive_streams_.push_back(new FakeVideoReceiveStream(config));
+    return video_receive_streams_[video_receive_streams_.size() - 1];
+  }
+
+  virtual void DestroyVideoReceiveStream(
+      webrtc::VideoReceiveStream* receive_stream) OVERRIDE {
+    FakeVideoReceiveStream* fake_stream =
+        static_cast<FakeVideoReceiveStream*>(receive_stream);
+    for (size_t i = 0; i < video_receive_streams_.size(); ++i) {
+      if (video_receive_streams_[i] == fake_stream) {
+        delete video_receive_streams_[i];
+        video_receive_streams_.erase(video_receive_streams_.begin() + i);
+        return;
+      }
+    }
+    ADD_FAILURE() << "DestroyVideoReceiveStream called with unknown paramter.";
+  }
+
+  virtual webrtc::PacketReceiver* Receiver() OVERRIDE {
+    // TODO(pbos): Fix this.
+    return NULL;
+  }
+
+  virtual uint32_t SendBitrateEstimate() OVERRIDE { return 0; }
+
+  virtual uint32_t ReceiveBitrateEstimate() OVERRIDE { return 0; }
+
+ private:
+  std::vector<webrtc::VideoCodec> codecs_;
+  std::vector<FakeVideoSendStream*> video_send_streams_;
+  std::vector<FakeVideoReceiveStream*> video_receive_streams_;
+};
+
+class FakeWebRtcVideoChannel2 : public WebRtcVideoChannel2 {
+ public:
+  FakeWebRtcVideoChannel2(FakeCall* call,
+                          WebRtcVideoEngine2* engine,
+                          VoiceMediaChannel* voice_channel)
+      : WebRtcVideoChannel2(call,
+                            engine,
+                            engine->GetDefaultVideoEncoderFactory()),
+        fake_call_(call),
+        voice_channel_(voice_channel) {}
+
+  virtual ~FakeWebRtcVideoChannel2() {}
+
+  VoiceMediaChannel* GetVoiceChannel() { return voice_channel_; }
+  FakeCall* GetFakeCall() { return fake_call_; }
+
+ private:
+  FakeCall* fake_call_;
+  VoiceMediaChannel* voice_channel_;
+};
+
+class FakeWebRtcVideoMediaChannelFactory : public WebRtcVideoChannelFactory {
+ public:
+  FakeWebRtcVideoChannel2* GetFakeChannel(VideoMediaChannel* channel) {
+    return channel_map_[channel];
+  }
+
+ private:
+  virtual WebRtcVideoChannel2* Create(WebRtcVideoEngine2* engine,
+                                      VoiceMediaChannel* voice_channel)
+      OVERRIDE {
+    FakeWebRtcVideoChannel2* channel =
+        new FakeWebRtcVideoChannel2(new FakeCall(), engine, voice_channel);
+    channel_map_[channel] = channel;
+    return channel;
+  }
+
+  std::map<VideoMediaChannel*, FakeWebRtcVideoChannel2*> channel_map_;
+};
+
+class WebRtcVideoEngine2Test : public testing::Test {
+ public:
+  WebRtcVideoEngine2Test()
+      : engine_(&factory_), engine_codecs_(engine_.codecs()) {
+    assert(!engine_codecs_.empty());
+    bool codec_set = false;
+    for (size_t i = 0; i < engine_codecs_.size(); ++i) {
+      if (engine_codecs_[i].name == "red") {
+        default_red_codec_ = engine_codecs_[i];
+      } else if (engine_codecs_[i].name == "ulpfec") {
+        default_ulpfec_codec_ = engine_codecs_[i];
+      } else if (engine_codecs_[i].name == "rtx") {
+        default_rtx_codec_ = engine_codecs_[i];
+      } else if (!codec_set) {
+        default_codec_ = engine_codecs_[i];
+        codec_set = true;
+      }
+    }
+
+    assert(codec_set);
+  }
+
+ protected:
+  FakeWebRtcVideoMediaChannelFactory factory_;
+  WebRtcVideoEngine2 engine_;
+  VideoCodec default_codec_;
+  VideoCodec default_red_codec_;
+  VideoCodec default_ulpfec_codec_;
+  VideoCodec default_rtx_codec_;
+  // TODO(pbos): Remove engine_codecs_ unless used a lot.
+  std::vector<VideoCodec> engine_codecs_;
+};
+
+TEST_F(WebRtcVideoEngine2Test, CreateChannel) {
+  talk_base::scoped_ptr<VideoMediaChannel> channel(engine_.CreateChannel(NULL));
+  ASSERT_TRUE(channel.get() != NULL) << "Could not create channel.";
+  EXPECT_TRUE(factory_.GetFakeChannel(channel.get()) != NULL)
+      << "Channel not created through factory.";
+}
+
+TEST_F(WebRtcVideoEngine2Test, CreateChannelWithVoiceEngine) {
+  VoiceMediaChannel* voice_channel = reinterpret_cast<VoiceMediaChannel*>(0x42);
+  talk_base::scoped_ptr<VideoMediaChannel> channel(
+      engine_.CreateChannel(voice_channel));
+  ASSERT_TRUE(channel.get() != NULL) << "Could not create channel.";
+
+  FakeWebRtcVideoChannel2* fake_channel =
+      factory_.GetFakeChannel(channel.get());
+  ASSERT_TRUE(fake_channel != NULL) << "Channel not created through factory.";
+
+  EXPECT_TRUE(fake_channel->GetVoiceChannel() != NULL)
+      << "VoiceChannel not set.";
+  EXPECT_EQ(voice_channel, fake_channel->GetVoiceChannel())
+      << "Different VoiceChannel set than the provided one.";
+}
+
+class WebRtcVideoChannel2BaseTest
+    : public VideoMediaChannelTest<WebRtcVideoEngine2, WebRtcVideoChannel2> {
+ protected:
+  virtual cricket::VideoCodec DefaultCodec() OVERRIDE { return kVp8Codec; }
+  typedef VideoMediaChannelTest<WebRtcVideoEngine2, WebRtcVideoChannel2> Base;
+};
+
+// TODO(pbos): Fix WebRtcVideoEngine2BaseTest, where we want CheckCoInitialize.
+#if 0
+// TODO(juberti): Figure out why ViE is munging the COM refcount.
+#ifdef WIN32
+TEST_F(WebRtcVideoChannel2BaseTest, DISABLED_CheckCoInitialize) {
+  Base::CheckCoInitialize();
+}
+#endif
+#endif
+
+TEST_F(WebRtcVideoChannel2BaseTest, SetSend) { Base::SetSend(); }
+
+TEST_F(WebRtcVideoChannel2BaseTest, SetSendWithoutCodecs) {
+  Base::SetSendWithoutCodecs();
+}
+
+TEST_F(WebRtcVideoChannel2BaseTest, SetSendSetsTransportBufferSizes) {
+  Base::SetSendSetsTransportBufferSizes();
+}
+
+// TODO(juberti): Fix this test to tolerate missing stats.
+TEST_F(WebRtcVideoChannel2BaseTest, DISABLED_GetStats) { Base::GetStats(); }
+
+// TODO(juberti): Fix this test to tolerate missing stats.
+TEST_F(WebRtcVideoChannel2BaseTest, DISABLED_GetStatsMultipleRecvStreams) {
+  Base::GetStatsMultipleRecvStreams();
+}
+
+TEST_F(WebRtcVideoChannel2BaseTest, DISABLED_GetStatsMultipleSendStreams) {
+  Base::GetStatsMultipleSendStreams();
+}
+
+TEST_F(WebRtcVideoChannel2BaseTest, SetSendBandwidth) {
+  Base::SetSendBandwidth();
+}
+TEST_F(WebRtcVideoChannel2BaseTest, SetSendSsrc) { Base::SetSendSsrc(); }
+TEST_F(WebRtcVideoChannel2BaseTest, SetSendSsrcAfterSetCodecs) {
+  Base::SetSendSsrcAfterSetCodecs();
+}
+
+TEST_F(WebRtcVideoChannel2BaseTest, SetRenderer) { Base::SetRenderer(); }
+
+TEST_F(WebRtcVideoChannel2BaseTest, AddRemoveRecvStreams) {
+  Base::AddRemoveRecvStreams();
+}
+
+TEST_F(WebRtcVideoChannel2BaseTest, DISABLED_AddRemoveRecvStreamAndRender) {
+  Base::AddRemoveRecvStreamAndRender();
+}
+
+TEST_F(WebRtcVideoChannel2BaseTest, AddRemoveRecvStreamsNoConference) {
+  Base::AddRemoveRecvStreamsNoConference();
+}
+
+TEST_F(WebRtcVideoChannel2BaseTest, AddRemoveSendStreams) {
+  Base::AddRemoveSendStreams();
+}
+
+TEST_F(WebRtcVideoChannel2BaseTest, SimulateConference) {
+  Base::SimulateConference();
+}
+
+TEST_F(WebRtcVideoChannel2BaseTest, AddRemoveCapturer) {
+  Base::AddRemoveCapturer();
+}
+
+TEST_F(WebRtcVideoChannel2BaseTest, RemoveCapturerWithoutAdd) {
+  Base::RemoveCapturerWithoutAdd();
+}
+
+TEST_F(WebRtcVideoChannel2BaseTest, AddRemoveCapturerMultipleSources) {
+  Base::AddRemoveCapturerMultipleSources();
+}
+
+// TODO(pbos): Figure out why this fails so often.
+TEST_F(WebRtcVideoChannel2BaseTest, DISABLED_HighAspectHighHeightCapturer) {
+  Base::HighAspectHighHeightCapturer();
+}
+
+TEST_F(WebRtcVideoChannel2BaseTest, RejectEmptyStreamParams) {
+  Base::RejectEmptyStreamParams();
+}
+
+TEST_F(WebRtcVideoChannel2BaseTest, AdaptResolution16x10) {
+  Base::AdaptResolution16x10();
+}
+
+TEST_F(WebRtcVideoChannel2BaseTest, AdaptResolution4x3) {
+  Base::AdaptResolution4x3();
+}
+
+TEST_F(WebRtcVideoChannel2BaseTest, MuteStream) { Base::MuteStream(); }
+
+TEST_F(WebRtcVideoChannel2BaseTest, MultipleSendStreams) {
+  Base::MultipleSendStreams();
+}
+
+// TODO(juberti): Restore this test once we support sending 0 fps.
+TEST_F(WebRtcVideoChannel2BaseTest, DISABLED_AdaptDropAllFrames) {
+  Base::AdaptDropAllFrames();
+}
+// TODO(juberti): Understand why we get decode errors on this test.
+TEST_F(WebRtcVideoChannel2BaseTest, DISABLED_AdaptFramerate) {
+  Base::AdaptFramerate();
+}
+
+TEST_F(WebRtcVideoChannel2BaseTest, SetSendStreamFormat0x0) {
+  Base::SetSendStreamFormat0x0();
+}
+
+// TODO(zhurunz): Fix the flakey test.
+TEST_F(WebRtcVideoChannel2BaseTest, DISABLED_SetSendStreamFormat) {
+  Base::SetSendStreamFormat();
+}
+
+TEST_F(WebRtcVideoChannel2BaseTest, TwoStreamsSendAndReceive) {
+  Base::TwoStreamsSendAndReceive(kVp8Codec);
+}
+
+TEST_F(WebRtcVideoChannel2BaseTest, TwoStreamsReUseFirstStream) {
+  Base::TwoStreamsReUseFirstStream(kVp8Codec);
+}
+
+class WebRtcVideoChannel2Test : public WebRtcVideoEngine2Test {
+ public:
+  virtual void SetUp() OVERRIDE {
+    channel_.reset(engine_.CreateChannel(NULL));
+    fake_channel_ = factory_.GetFakeChannel(channel_.get());
+    last_ssrc_ = 123;
+    ASSERT_TRUE(fake_channel_ != NULL)
+        << "Channel not created through factory.";
+  }
+
+ protected:
+  FakeVideoSendStream* AddSendStream() {
+    return AddSendStream(StreamParams::CreateLegacy(last_ssrc_++));
+  }
+
+  FakeVideoSendStream* AddSendStream(const StreamParams& sp) {
+    size_t num_streams =
+        fake_channel_->GetFakeCall()->GetVideoSendStreams().size();
+    EXPECT_TRUE(channel_->AddSendStream(sp));
+    std::vector<FakeVideoSendStream*> streams =
+        fake_channel_->GetFakeCall()->GetVideoSendStreams();
+    EXPECT_EQ(num_streams + 1, streams.size());
+    return streams[streams.size() - 1];
+  }
+
+  std::vector<FakeVideoSendStream*> GetFakeSendStreams() {
+    return fake_channel_->GetFakeCall()->GetVideoSendStreams();
+  }
+
+  FakeVideoReceiveStream* AddRecvStream() {
+    return AddRecvStream(StreamParams::CreateLegacy(last_ssrc_++));
+  }
+
+  FakeVideoReceiveStream* AddRecvStream(const StreamParams& sp) {
+    size_t num_streams =
+        fake_channel_->GetFakeCall()->GetVideoReceiveStreams().size();
+    EXPECT_TRUE(channel_->AddRecvStream(sp));
+    std::vector<FakeVideoReceiveStream*> streams =
+        fake_channel_->GetFakeCall()->GetVideoReceiveStreams();
+    EXPECT_EQ(num_streams + 1, streams.size());
+    return streams[streams.size() - 1];
+  }
+
+  void SetSendCodecsShouldWorkForBitrates(const char* min_bitrate,
+                                          const char* max_bitrate) {
+    std::vector<VideoCodec> codecs;
+    codecs.push_back(kVp8Codec);
+    codecs[0].params[kCodecParamMinBitrate] = min_bitrate;
+    codecs[0].params[kCodecParamMaxBitrate] = max_bitrate;
+    EXPECT_TRUE(channel_->SetSendCodecs(codecs));
+
+    FakeVideoSendStream* stream = AddSendStream();
+
+    webrtc::VideoSendStream::Config::EncoderSettings encoder_settings =
+        stream->GetConfig().encoder_settings;
+    ASSERT_EQ(1u, encoder_settings.streams.size());
+    EXPECT_EQ(atoi(min_bitrate),
+              encoder_settings.streams.back().min_bitrate_bps / 1000);
+    EXPECT_EQ(atoi(max_bitrate),
+              encoder_settings.streams.back().max_bitrate_bps / 1000);
+
+    VideoCodec codec;
+    EXPECT_TRUE(channel_->GetSendCodec(&codec));
+    EXPECT_EQ(min_bitrate, codec.params[kCodecParamMinBitrate]);
+    EXPECT_EQ(max_bitrate, codec.params[kCodecParamMaxBitrate]);
+  }
+
+  void ExpectEqualCodecs(const VideoCodec video_codec,
+                         const webrtc::VideoCodec& webrtc_codec) {
+    EXPECT_STREQ(video_codec.name.c_str(), webrtc_codec.plName);
+    EXPECT_EQ(video_codec.id, webrtc_codec.plType);
+    EXPECT_EQ(video_codec.width, webrtc_codec.width);
+    EXPECT_EQ(video_codec.height, webrtc_codec.height);
+    EXPECT_EQ(video_codec.framerate, webrtc_codec.maxFramerate);
+  }
+  talk_base::scoped_ptr<VideoMediaChannel> channel_;
+  FakeWebRtcVideoChannel2* fake_channel_;
+  uint32 last_ssrc_;
+};
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_MaxBitrateResetsWithConferenceMode) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_StartSendBitrate) {
+  // TODO(pbos): Is this test testing vie_ ? this is confusing. No API to set
+  // start send bitrate from outside? Add defaults here that should be kept?
+  std::vector<cricket::VideoCodec> codec_list;
+  codec_list.push_back(kVp8Codec);
+  EXPECT_TRUE(channel_->SetSendCodecs(codec_list));
+  const unsigned int kVideoMinSendBitrateKbps = 50;
+  const unsigned int kVideoTargetSendBitrateKbps = 300;
+  const unsigned int kVideoMaxSendBitrateKbps = 2000;
+  FakeVideoSendStream* stream = AddSendStream();
+  webrtc::VideoSendStream::Config::EncoderSettings encoder_settings =
+      stream->GetConfig().encoder_settings;
+  ASSERT_EQ(1u, encoder_settings.streams.size());
+  EXPECT_EQ(kVideoMinSendBitrateKbps,
+            encoder_settings.streams.back().min_bitrate_bps / 1000);
+  EXPECT_EQ(kVideoTargetSendBitrateKbps,
+            encoder_settings.streams.back().target_bitrate_bps / 1000);
+  EXPECT_EQ(kVideoMaxSendBitrateKbps,
+            encoder_settings.streams.back().max_bitrate_bps / 1000);
+#if 0
+  // TODO(pbos): un-#if
+  VerifyVP8SendCodec(send_channel, kVP8Codec.width, kVP8Codec.height, 0,
+                     kVideoMaxSendBitrateKbps, kVideoMinSendBitrateKbps,
+                     kVideoDefaultStartSendBitrateKbps);
+  EXPECT_EQ(0, vie_.StartSend(send_channel));
+
+  // Increase the send bitrate and verify it is used as start bitrate.
+  const unsigned int kVideoSendBitrateBps = 768000;
+  vie_.SetSendBitrates(send_channel, kVideoSendBitrateBps, 0, 0);
+  EXPECT_TRUE(channel_->SetSendCodecs(codec_list));
+  VerifyVP8SendCodec(send_channel, kVP8Codec.width, kVP8Codec.height, 0,
+                     kVideoMaxSendBitrateKbps, kVideoMinSendBitrateKbps,
+                     kVideoSendBitrateBps / 1000);
+
+  // Never set a start bitrate higher than the max bitrate.
+  vie_.SetSendBitrates(send_channel, kVideoMaxSendBitrateKbps + 500, 0, 0);
+  EXPECT_TRUE(channel_->SetSendCodecs(codec_list));
+  VerifyVP8SendCodec(send_channel, kVP8Codec.width, kVP8Codec.height, 0,
+                     kVideoMaxSendBitrateKbps, kVideoMinSendBitrateKbps,
+                     kVideoDefaultStartSendBitrateKbps);
+
+  // Use the default start bitrate if the send bitrate is lower.
+  vie_.SetSendBitrates(send_channel, kVideoDefaultStartSendBitrateKbps - 50, 0,
+                       0);
+  EXPECT_TRUE(channel_->SetSendCodecs(codec_list));
+  VerifyVP8SendCodec(send_channel, kVP8Codec.width, kVP8Codec.height, 0,
+                     kVideoMaxSendBitrateKbps, kVideoMinSendBitrateKbps,
+                     kVideoDefaultStartSendBitrateKbps);
+#endif
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_RtcpEnabled) {
+  // Note(pbos): This is a receiver-side setting, dumbo.
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_KeyFrameRequestEnabled) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, RembIsEnabledByDefault) {
+  FakeVideoReceiveStream* stream = AddRecvStream();
+  EXPECT_TRUE(stream->GetConfig().rtp.remb);
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_RembEnabledOnReceiveChannels) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, RecvStreamWithRtx) {
+  EXPECT_TRUE(channel_->SetSendCodecs(engine_.codecs()));
+  EXPECT_TRUE(channel_->SetSend(true));
+  cricket::VideoOptions options;
+  options.conference_mode.Set(true);
+  EXPECT_TRUE(channel_->SetOptions(options));
+
+  // Send side.
+  const std::vector<uint32> ssrcs = MAKE_VECTOR(kSsrcs1);
+  const std::vector<uint32> rtx_ssrcs = MAKE_VECTOR(kRtxSsrcs1);
+  FakeVideoSendStream* send_stream = AddSendStream(
+      cricket::CreateSimWithRtxStreamParams("cname", ssrcs, rtx_ssrcs));
+
+  ASSERT_EQ(rtx_ssrcs.size(), send_stream->GetConfig().rtp.rtx.ssrcs.size());
+  for (size_t i = 0; i < rtx_ssrcs.size(); ++i)
+    EXPECT_EQ(rtx_ssrcs[i], send_stream->GetConfig().rtp.rtx.ssrcs[i]);
+
+  // Receiver side.
+  FakeVideoReceiveStream* recv_stream = AddRecvStream(
+      cricket::CreateSimWithRtxStreamParams("cname", ssrcs, rtx_ssrcs));
+  ASSERT_GT(recv_stream->GetConfig().rtp.rtx.size(), 0u)
+      << "No SSRCs for RTX configured by AddRecvStream.";
+  ASSERT_EQ(1u, recv_stream->GetConfig().rtp.rtx.size())
+      << "This test only works with one receive codec. Please update the test.";
+  EXPECT_EQ(rtx_ssrcs[0],
+            recv_stream->GetConfig().rtp.rtx.begin()->second.ssrc);
+  // TODO(pbos): Make sure we set the RTX for correct payloads etc.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_RecvStreamWithRtxOnMultiplePayloads) {
+  FAIL() << "Not implemented.";
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_RecvStreamNoRtx) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_RtpTimestampOffsetHeaderExtensions) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_AbsoluteSendTimeHeaderExtensions) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_LeakyBucketTest) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_BufferedModeLatency) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_AdditiveVideoOptions) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, AddRecvStreamOnlyUsesOneReceiveStream) {
+  EXPECT_TRUE(channel_->AddRecvStream(cricket::StreamParams::CreateLegacy(1)));
+  EXPECT_EQ(1u, fake_channel_->GetFakeCall()->GetVideoReceiveStreams().size());
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_NoRembChangeAfterAddRecvStream) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_RembOnOff) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_NackEnabled) {
+  // Verify NACK on both sender and receiver.
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_VideoProtectionInterop) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_VideoProtectionInteropReversed) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_HybridNackFecConference) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_AddRemoveRecvStreamConference) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_SetRender) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_SetBandwidthAuto) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_SetBandwidthAutoCapped) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_SetBandwidthFixed) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_SetBandwidthInConference) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_SetBandwidthScreencast) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_SetSendSsrcAndCname) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test,
+       DISABLED_SetSendSsrcAfterCreatingReceiveChannel) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_SetOptionsWithDenoising) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_MultipleSendStreamsWithOneCapturer) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_DISABLED_SendReceiveBitratesStats) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_TestSetAdaptInputToCpuUsage) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_TestSetCpuThreshold) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_TestSetInvalidCpuThreshold) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_WebRtcShouldLog) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_WebRtcShouldNotLog) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoEngine2Test, FindCodec) {
+  const std::vector<cricket::VideoCodec>& c = engine_.codecs();
+  EXPECT_EQ(4U, c.size());
+
+  cricket::VideoCodec vp8(104, "VP8", 320, 200, 30, 0);
+  EXPECT_TRUE(engine_.FindCodec(vp8));
+
+  cricket::VideoCodec vp8_ci(104, "vp8", 320, 200, 30, 0);
+  EXPECT_TRUE(engine_.FindCodec(vp8));
+
+  cricket::VideoCodec vp8_diff_fr_diff_pref(104, "VP8", 320, 200, 50, 50);
+  EXPECT_TRUE(engine_.FindCodec(vp8_diff_fr_diff_pref));
+
+  cricket::VideoCodec vp8_diff_id(95, "VP8", 320, 200, 30, 0);
+  EXPECT_FALSE(engine_.FindCodec(vp8_diff_id));
+  vp8_diff_id.id = 97;
+  EXPECT_TRUE(engine_.FindCodec(vp8_diff_id));
+
+  cricket::VideoCodec vp8_diff_res(104, "VP8", 320, 111, 30, 0);
+  EXPECT_FALSE(engine_.FindCodec(vp8_diff_res));
+
+  // PeerConnection doesn't negotiate the resolution at this point.
+  // Test that FindCodec can handle the case when width/height is 0.
+  cricket::VideoCodec vp8_zero_res(104, "VP8", 0, 0, 30, 0);
+  EXPECT_TRUE(engine_.FindCodec(vp8_zero_res));
+
+  cricket::VideoCodec red(101, "RED", 0, 0, 30, 0);
+  EXPECT_TRUE(engine_.FindCodec(red));
+
+  cricket::VideoCodec red_ci(101, "red", 0, 0, 30, 0);
+  EXPECT_TRUE(engine_.FindCodec(red));
+
+  cricket::VideoCodec fec(102, "ULPFEC", 0, 0, 30, 0);
+  EXPECT_TRUE(engine_.FindCodec(fec));
+
+  cricket::VideoCodec fec_ci(102, "ulpfec", 0, 0, 30, 0);
+  EXPECT_TRUE(engine_.FindCodec(fec));
+
+  cricket::VideoCodec rtx(96, "rtx", 0, 0, 30, 0);
+  EXPECT_TRUE(engine_.FindCodec(rtx));
+}
+
+TEST_F(WebRtcVideoEngine2Test, DefaultRtxCodecHasAssociatedPayloadTypeSet) {
+  for (size_t i = 0; i < engine_codecs_.size(); ++i) {
+    if (engine_codecs_[i].name != kRtxCodecName)
+      continue;
+    int associated_payload_type;
+    EXPECT_TRUE(engine_codecs_[i].GetParam(kCodecParamAssociatedPayloadType,
+                                           &associated_payload_type));
+    EXPECT_EQ(default_codec_.id, associated_payload_type);
+    return;
+  }
+  FAIL() << "No RTX codec found among default codecs.";
+}
+
+TEST_F(WebRtcVideoChannel2Test, SetDefaultSendCodecs) {
+  ASSERT_TRUE(channel_->SetSendCodecs(engine_codecs_));
+
+  VideoCodec codec;
+  EXPECT_TRUE(channel_->GetSendCodec(&codec));
+  EXPECT_TRUE(codec.Matches(engine_codecs_[0]));
+
+  // Using a RTX setup to verify that the default RTX payload type is good.
+  const std::vector<uint32> ssrcs = MAKE_VECTOR(kSsrcs1);
+  const std::vector<uint32> rtx_ssrcs = MAKE_VECTOR(kRtxSsrcs1);
+  FakeVideoSendStream* stream = AddSendStream(
+      cricket::CreateSimWithRtxStreamParams("cname", ssrcs, rtx_ssrcs));
+  webrtc::VideoSendStream::Config config = stream->GetConfig();
+  // TODO(pbos): Replace ExpectEqualCodecs.
+  // ExpectEqualCodecs(engine_codecs_[0], config.codec);
+
+  // Make sure NACK and FEC are enabled on the correct payload types.
+  EXPECT_EQ(1000, config.rtp.nack.rtp_history_ms);
+  EXPECT_EQ(default_ulpfec_codec_.id, config.rtp.fec.ulpfec_payload_type);
+  EXPECT_EQ(default_red_codec_.id, config.rtp.fec.red_payload_type);
+  // TODO(pbos): Verify that the rtx ssrc is set, correct, not taken by anything
+  //             else.
+  // ASSERT_EQ(1u, config.rtp.rtx.ssrcs.size());
+  EXPECT_EQ(static_cast<int>(default_rtx_codec_.id),
+            config.rtp.rtx.payload_type);
+  // TODO(juberti): Check RTCP, PLI, TMMBR.
+}
+
+TEST_F(WebRtcVideoChannel2Test, SetSendCodecsWithoutFec) {
+  std::vector<VideoCodec> codecs;
+  codecs.push_back(kVp8Codec);
+  ASSERT_TRUE(channel_->SetSendCodecs(codecs));
+
+  FakeVideoSendStream* stream = AddSendStream();
+  webrtc::VideoSendStream::Config config = stream->GetConfig();
+
+  EXPECT_EQ(-1, config.rtp.fec.ulpfec_payload_type);
+  EXPECT_EQ(-1, config.rtp.fec.red_payload_type);
+}
+
+TEST_F(WebRtcVideoChannel2Test,
+       DISABLED_SetSendCodecRejectsRtxWithoutAssociatedPayloadType) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test,
+       DISABLED_SetSendCodecRejectsRtxWithoutMatchingVideoCodec) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test,
+       DISABLED_SetCodecsWithoutFecDisablesCurrentFec) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_SetSendCodecsChangesExistingStreams) {
+  FAIL();  // TODO(pbos): Implement, make sure that it's changing running
+           //             streams. Should it?
+}
+
+TEST_F(WebRtcVideoChannel2Test,
+       DISABLED_ConstrainsSetCodecsAccordingToEncoderConfig) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, SetSendCodecsWithMinMaxBitrate) {
+  SetSendCodecsShouldWorkForBitrates("10", "20");
+}
+
+TEST_F(WebRtcVideoChannel2Test, SetSendCodecsRejectsMaxLessThanMinBitrate) {
+  engine_codecs_[0].params[kCodecParamMinBitrate] = "30";
+  engine_codecs_[0].params[kCodecParamMaxBitrate] = "20";
+  EXPECT_FALSE(channel_->SetSendCodecs(engine_codecs_));
+}
+
+TEST_F(WebRtcVideoChannel2Test, SetSendCodecsAcceptLargeMinMaxBitrate) {
+  SetSendCodecsShouldWorkForBitrates("1000", "2000");
+}
+
+TEST_F(WebRtcVideoChannel2Test, SetSendCodecsWithMaxQuantization) {
+  static const char* kMaxQuantization = "21";
+  std::vector<VideoCodec> codecs;
+  codecs.push_back(kVp8Codec);
+  codecs[0].params[kCodecParamMaxQuantization] = kMaxQuantization;
+  EXPECT_TRUE(channel_->SetSendCodecs(codecs));
+  EXPECT_EQ(
+      static_cast<unsigned int>(atoi(kMaxQuantization)),
+      AddSendStream()->GetConfig().encoder_settings.streams.back().max_qp);
+
+  VideoCodec codec;
+  EXPECT_TRUE(channel_->GetSendCodec(&codec));
+  EXPECT_EQ(kMaxQuantization, codec.params[kCodecParamMaxQuantization]);
+}
+
+TEST_F(WebRtcVideoChannel2Test, SetSendCodecsRejectBadDimensions) {
+  std::vector<cricket::VideoCodec> codecs;
+  codecs.push_back(kVp8Codec);
+
+  codecs[0].width = 0;
+  EXPECT_FALSE(channel_->SetSendCodecs(codecs))
+      << "Codec set though codec width is zero.";
+
+  codecs[0].width = kVp8Codec.width;
+  codecs[0].height = 0;
+  EXPECT_FALSE(channel_->SetSendCodecs(codecs))
+      << "Codec set though codec height is zero.";
+}
+
+TEST_F(WebRtcVideoChannel2Test, SetSendCodecsRejectBadPayloadTypes) {
+  // TODO(pbos): Should we only allow the dynamic range?
+  static const size_t kNumIncorrectPayloads = 4;
+  static const int kIncorrectPayloads[kNumIncorrectPayloads] = {-2, -1, 128,
+                                                                129};
+  std::vector<cricket::VideoCodec> codecs;
+  codecs.push_back(kVp8Codec);
+  for (size_t i = 0; i < kNumIncorrectPayloads; ++i) {
+    int payload_type = kIncorrectPayloads[i];
+    codecs[0].id = payload_type;
+    EXPECT_FALSE(channel_->SetSendCodecs(codecs))
+        << "Bad payload type '" << payload_type << "' accepted.";
+  }
+}
+
+TEST_F(WebRtcVideoChannel2Test, SetSendCodecsAcceptAllValidPayloadTypes) {
+  std::vector<cricket::VideoCodec> codecs;
+  codecs.push_back(kVp8Codec);
+  for (int payload_type = 0; payload_type <= 127; ++payload_type) {
+    codecs[0].id = payload_type;
+    EXPECT_TRUE(channel_->SetSendCodecs(codecs))
+        << "Payload type '" << payload_type << "' rejected.";
+  }
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_ResetVieSendCodecOnNewFrameSize) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, SetRecvCodecsWithOnlyVp8) {
+  std::vector<cricket::VideoCodec> codecs;
+  codecs.push_back(kVp8Codec);
+  EXPECT_TRUE(channel_->SetRecvCodecs(codecs));
+}
+
+TEST_F(WebRtcVideoChannel2Test, SetRecvCodecsDifferentPayloadType) {
+  std::vector<cricket::VideoCodec> codecs;
+  codecs.push_back(kVp8Codec);
+  codecs[0].id = 99;
+  EXPECT_TRUE(channel_->SetRecvCodecs(codecs));
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_SetRecvCodecsAcceptDefaultCodecs) {
+  EXPECT_TRUE(channel_->SetRecvCodecs(engine_.codecs()));
+  // (I've added this one.) Make sure they propagate down to VideoReceiveStream!
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, SetRecvCodecsRejectUnsupportedCodec) {
+  std::vector<VideoCodec> codecs;
+  codecs.push_back(kVp8Codec);
+  codecs.push_back(VideoCodec(101, "WTF3", 640, 400, 30, 0));
+  EXPECT_FALSE(channel_->SetRecvCodecs(codecs));
+}
+
+// TODO(pbos): Enable VP9 through external codec support
+TEST_F(WebRtcVideoChannel2Test,
+       DISABLED_SetRecvCodecsAcceptsMultipleVideoCodecs) {
+  std::vector<VideoCodec> codecs;
+  codecs.push_back(kVp8Codec);
+  codecs.push_back(kVp9Codec);
+  EXPECT_TRUE(channel_->SetRecvCodecs(codecs));
+}
+
+TEST_F(WebRtcVideoChannel2Test,
+       DISABLED_SetRecvCodecsSetsFecForAllVideoCodecs) {
+  std::vector<VideoCodec> codecs;
+  codecs.push_back(kVp8Codec);
+  codecs.push_back(kVp9Codec);
+  EXPECT_TRUE(channel_->SetRecvCodecs(codecs));
+  FAIL();  // TODO(pbos): Verify that the FEC parameters are set for all codecs.
+}
+
+TEST_F(WebRtcVideoChannel2Test, SetSendCodecsRejectDuplicateFecPayloads) {
+  std::vector<VideoCodec> codecs;
+  codecs.push_back(kVp8Codec);
+  codecs.push_back(kRedCodec);
+  codecs[1].id = codecs[0].id;
+  EXPECT_FALSE(channel_->SetRecvCodecs(codecs));
+}
+
+TEST_F(WebRtcVideoChannel2Test, SetRecvCodecsRejectDuplicateCodecPayloads) {
+  std::vector<VideoCodec> codecs;
+  codecs.push_back(kVp8Codec);
+  codecs.push_back(kVp9Codec);
+  codecs[1].id = codecs[0].id;
+  EXPECT_FALSE(channel_->SetRecvCodecs(codecs));
+}
+
+TEST_F(WebRtcVideoChannel2Test,
+       SetRecvCodecsAcceptSameCodecOnMultiplePayloadTypes) {
+  std::vector<VideoCodec> codecs;
+  codecs.push_back(kVp8Codec);
+  codecs.push_back(kVp8Codec);
+  codecs[1].id += 1;
+  EXPECT_TRUE(channel_->SetRecvCodecs(codecs));
+}
+
+TEST_F(WebRtcVideoChannel2Test, SendStreamNotSendingByDefault) {
+  EXPECT_FALSE(AddSendStream()->IsSending());
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_ReceiveStreamReceivingByDefault) {
+  // Is this test correct though? Auto-receive? Enable receive on first packet?
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, SetSend) {
+  AddSendStream();
+  EXPECT_FALSE(channel_->SetSend(true))
+      << "Channel should not start without codecs.";
+  EXPECT_TRUE(channel_->SetSend(false))
+      << "Channel should be stoppable even without set codecs.";
+
+  std::vector<cricket::VideoCodec> codecs;
+  codecs.push_back(kVp8Codec);
+  channel_->SetSendCodecs(codecs);
+  std::vector<FakeVideoSendStream*> streams = GetFakeSendStreams();
+  ASSERT_EQ(1u, streams.size());
+  FakeVideoSendStream* stream = streams.back();
+
+  EXPECT_FALSE(stream->IsSending());
+
+  // false->true
+  EXPECT_TRUE(channel_->SetSend(true));
+  EXPECT_TRUE(stream->IsSending());
+  // true->true
+  EXPECT_TRUE(channel_->SetSend(true));
+  EXPECT_TRUE(stream->IsSending());
+  // true->false
+  EXPECT_TRUE(channel_->SetSend(false));
+  EXPECT_FALSE(stream->IsSending());
+  // false->false
+  EXPECT_TRUE(channel_->SetSend(false));
+  EXPECT_FALSE(stream->IsSending());
+
+  EXPECT_TRUE(channel_->SetSend(true));
+  FakeVideoSendStream* new_stream = AddSendStream();
+  EXPECT_TRUE(new_stream->IsSending())
+      << "Send stream created after SetSend(true) not sending initially.";
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_SendAndReceiveVp8Vga) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_SendAndReceiveVp8Qvga) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_SendAndReceiveH264SvcQqvga) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_SendManyResizeOnce) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_SendVp8HdAndReceiveAdaptedVp8Vga) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_TestSetDscpOptions) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_SetOptionsWithMaxBitrate) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_SetOptionsWithLoweredBitrate) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_SetOptionsSucceedsWhenSending) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_ResetCodecOnScreencast) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_DontResetCodecOnSendFrame) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test,
+       DISABLED_DontRegisterDecoderIfFactoryIsNotGiven) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_RegisterDecoderIfFactoryIsGiven) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_DontRegisterDecoderMultipleTimes) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_DontRegisterDecoderForNonVP8) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test,
+       DISABLED_DontRegisterEncoderIfFactoryIsNotGiven) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_RegisterEncoderIfFactoryIsGiven) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_DontRegisterEncoderMultipleTimes) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test,
+       DISABLED_RegisterEncoderWithMultipleSendStreams) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_DontRegisterEncoderForNonVP8) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_FeedbackParamsForNonVP8) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_ExternalCodecAddedToTheEnd) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_ExternalCodecIgnored) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_UpdateEncoderCodecsAfterSetFactory) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_OnReadyToSend) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+
+TEST_F(WebRtcVideoChannel2Test, DISABLED_CaptureFrameTimestampToNtpTimestamp) {
+  FAIL() << "Not implemented.";  // TODO(pbos): Implement.
+}
+}  // namespace cricket