Remove webrtc::VideoEncoderFactory

webrtc/video/picture_id_tests.cc

 /*
  *  Copyright (c) 2013 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 #include "webrtc/media/engine/internalencoderfactory.h"
+#include "webrtc/media/engine/simulcast_encoder_adapter.h"
 #include "webrtc/modules/rtp_rtcp/source/rtp_format.h"
-#include "webrtc/modules/video_coding/codecs/vp8/simulcast_encoder_adapter.h"
 #include "webrtc/modules/video_coding/sequence_number_util.h"
 #include "webrtc/test/call_test.h"
 
 namespace webrtc {
 
 const int kFrameMaxWidth = 1280;
 const int kFrameMaxHeight = 720;
 const int kFrameRate = 30;
 const int kMaxSecondsLost = 5;
 const int kMaxFramesLost = kFrameRate * kMaxSecondsLost;
@@ skipping 261 matching lines @@
 
 TEST_F(PictureIdTest, PictureIdIncreasingAfterStreamCountChangeVp8) {
   std::unique_ptr<VideoEncoder> encoder(VP8Encoder::Create());
   // Make sure that that the picture id is not reset if the stream count goes
   // down and then up.
   std::vector<int> ssrc_counts = {3, 1, 3};
   SetupEncoder(encoder.get());
   TestPictureIdContinuousAfterReconfigure(ssrc_counts);
 }
 
-class VideoEncoderFactoryAdapter : public webrtc::VideoEncoderFactory {
- public:
-  explicit VideoEncoderFactoryAdapter(
-      cricket::WebRtcVideoEncoderFactory* factory)
-      : factory_(factory) {}
-  virtual ~VideoEncoderFactoryAdapter() {}
-
-  // Implements webrtc::VideoEncoderFactory.
-  webrtc::VideoEncoder* Create() override {
-    return factory_->CreateVideoEncoder(
-        cricket::VideoCodec(cricket::kVp8CodecName));
-  }
-
-  void Destroy(webrtc::VideoEncoder* encoder) override {
-    return factory_->DestroyVideoEncoder(encoder);
-  }
-
- private:
-  cricket::WebRtcVideoEncoderFactory* const factory_;
-};
-
 TEST_F(PictureIdTest,
        PictureIdContinuousAfterReconfigureSimulcastEncoderAdapter) {
   cricket::InternalEncoderFactory internal_encoder_factory;
-  SimulcastEncoderAdapter simulcast_encoder_adapter(
-      new VideoEncoderFactoryAdapter(&internal_encoder_factory));
+  SimulcastEncoderAdapter simulcast_encoder_adapter(&internal_encoder_factory);
   SetupEncoder(&simulcast_encoder_adapter);
   TestPictureIdContinuousAfterReconfigure({1, 3, 3, 1, 1});
 }
 
 TEST_F(PictureIdTest,
        PictureIdIncreasingAfterRecreateStreamSimulcastEncoderAdapter) {
   cricket::InternalEncoderFactory internal_encoder_factory;
-  SimulcastEncoderAdapter simulcast_encoder_adapter(
-      new VideoEncoderFactoryAdapter(&internal_encoder_factory));
+  SimulcastEncoderAdapter simulcast_encoder_adapter(&internal_encoder_factory);
   SetupEncoder(&simulcast_encoder_adapter);
   TestPictureIdIncreaseAfterRecreateStreams({1, 3, 3, 1, 1});
 }
 
 // When using the simulcast encoder adapter, the picture id is randomly set
 // when the ssrc count is reduced and then increased. This means that we are
 // not spec compliant in that particular case.
 TEST_F(
     PictureIdTest,
     DISABLED_PictureIdIncreasingAfterStreamCountChangeSimulcastEncoderAdapter) {
   cricket::InternalEncoderFactory internal_encoder_factory;
-  SimulcastEncoderAdapter simulcast_encoder_adapter(
-      new VideoEncoderFactoryAdapter(&internal_encoder_factory));
+  SimulcastEncoderAdapter simulcast_encoder_adapter(&internal_encoder_factory);
   // Make sure that that the picture id is not reset if the stream count goes
   // down and then up.
   std::vector<int> ssrc_counts = {3, 1, 3};
   SetupEncoder(&simulcast_encoder_adapter);
   TestPictureIdContinuousAfterReconfigure(ssrc_counts);
 }
 
 }  // namespace webrtc