Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(164)

Unified Diff: webrtc/media/engine/webrtcvideoencoderfactory.cc

Issue 2449993003: Replace WebRtcVideoEncoderFactory::VideoCodec with cricket::VideoCodec (Closed)
Patch Set: Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webrtc/media/engine/webrtcvideoencoderfactory.cc
diff --git a/webrtc/media/engine/webrtcvideoencoderfactory.cc b/webrtc/media/engine/webrtcvideoencoderfactory.cc
new file mode 100644
index 0000000000000000000000000000000000000000..ccc32d894bf2e19e336b972b04abca212e95424f
--- /dev/null
+++ b/webrtc/media/engine/webrtcvideoencoderfactory.cc
@@ -0,0 +1,60 @@
+/*
+ * Copyright (c) 2016 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/webrtcvideoencoderfactory.h"
+
+#include "webrtc/media/base/mediaconstants.h"
+
+static const char* NameFromCodecType(webrtc::VideoCodecType type) {
+ switch (type) {
+ case webrtc::kVideoCodecVP8:
+ return cricket::kVp8CodecName;
+ case webrtc::kVideoCodecVP9:
+ return cricket::kVp9CodecName;
+ case webrtc::kVideoCodecH264:
+ return cricket::kH264CodecName;
+ default:
+ return "Unknown codec";
+ }
+}
+
+namespace cricket {
+
+webrtc::VideoEncoder* WebRtcVideoEncoderFactory::CreateVideoEncoder(
+ const cricket::VideoCodec& codec) {
+ return CreateVideoEncoder(CodecTypeFromName(codec.name));
+}
+
+const std::vector<cricket::VideoCodec>&
+WebRtcVideoEncoderFactory::supported_codecs() const {
+ const std::vector<VideoCodec>& encoder_codecs = codecs();
+ for (const VideoCodec& encoder_codec : encoder_codecs) {
+ codecs_.push_back(cricket::VideoCodec(encoder_codec.name));
+ }
+ return codecs_;
+}
+
+webrtc::VideoEncoder* WebRtcVideoEncoderFactory::CreateVideoEncoder(
+ webrtc::VideoCodecType type) {
+ const cricket::VideoCodec codec(NameFromCodecType(type));
+ return CreateVideoEncoder(codec);
+}
+
+const std::vector<WebRtcVideoEncoderFactory::VideoCodec>&
+WebRtcVideoEncoderFactory::codecs() const {
+ const std::vector<cricket::VideoCodec>& codecs = supported_codecs();
+ for (const cricket::VideoCodec& codec : codecs) {
+ encoder_codecs_.push_back(
+ VideoCodec(CodecTypeFromName(codec.name), codec.name));
+ }
+ return encoder_codecs_;
+}
+
+} // namespace cricket

Powered by Google App Engine
This is Rietveld 408576698