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

Unified Diff: content/renderer/media/rtc_video_encoder_factory.cc

Issue 2095393002: Add posciak and wuchengli to WebRTC HW decoder and encoder files (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: remove duplicated owners Created 4 years, 6 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: content/renderer/media/rtc_video_encoder_factory.cc
diff --git a/content/renderer/media/rtc_video_encoder_factory.cc b/content/renderer/media/rtc_video_encoder_factory.cc
deleted file mode 100644
index 7e99b7c13865ac2358e374554adf50c0ec4ce2b7..0000000000000000000000000000000000000000
--- a/content/renderer/media/rtc_video_encoder_factory.cc
+++ /dev/null
@@ -1,81 +0,0 @@
-// Copyright 2013 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "content/renderer/media/rtc_video_encoder_factory.h"
-
-#include "base/command_line.h"
-#include "content/public/common/content_switches.h"
-#include "content/public/common/feature_h264_with_openh264_ffmpeg.h"
-#include "content/renderer/media/rtc_video_encoder.h"
-#include "media/gpu/ipc/client/gpu_video_encode_accelerator_host.h"
-#include "media/renderers/gpu_video_accelerator_factories.h"
-#include "media/video/video_encode_accelerator.h"
-
-namespace content {
-
-namespace {
-
-// Translate from media::VideoEncodeAccelerator::SupportedProfile to
-// one or more instances of cricket::WebRtcVideoEncoderFactory::VideoCodec
-void VEAToWebRTCCodecs(
- std::vector<cricket::WebRtcVideoEncoderFactory::VideoCodec>* codecs,
- const media::VideoEncodeAccelerator::SupportedProfile& profile) {
- const int width = profile.max_resolution.width();
- const int height = profile.max_resolution.height();
- const int fps = profile.max_framerate_numerator;
- DCHECK_EQ(profile.max_framerate_denominator, 1U);
-
- const base::CommandLine* cmd_line = base::CommandLine::ForCurrentProcess();
- if (profile.profile >= media::VP8PROFILE_MIN &&
- profile.profile <= media::VP8PROFILE_MAX) {
- codecs->push_back(cricket::WebRtcVideoEncoderFactory::VideoCodec(
- webrtc::kVideoCodecVP8, "VP8", width, height, fps));
- } else if (profile.profile >= media::H264PROFILE_MIN &&
- profile.profile <= media::H264PROFILE_MAX) {
- bool webrtc_h264_enabled = false;
-#if BUILDFLAG(RTC_USE_H264) && defined(OS_MACOSX)
- webrtc_h264_enabled =
- base::FeatureList::IsEnabled(kWebRtcH264WithOpenH264FFmpeg);
-#endif // BUILDFLAG(RTC_USE_H264) && defined(OS_MACOSX)
- if (cmd_line->HasSwitch(switches::kEnableWebRtcHWH264Encoding) ||
- webrtc_h264_enabled) {
- codecs->push_back(cricket::WebRtcVideoEncoderFactory::VideoCodec(
- webrtc::kVideoCodecH264, "H264", width, height, fps));
- }
- }
-}
-
-} // anonymous namespace
-
-RTCVideoEncoderFactory::RTCVideoEncoderFactory(
- media::GpuVideoAcceleratorFactories* gpu_factories)
- : gpu_factories_(gpu_factories) {
- const media::VideoEncodeAccelerator::SupportedProfiles& profiles =
- gpu_factories_->GetVideoEncodeAcceleratorSupportedProfiles();
- for (const auto& profile : profiles)
- VEAToWebRTCCodecs(&codecs_, profile);
-}
-
-RTCVideoEncoderFactory::~RTCVideoEncoderFactory() {}
-
-webrtc::VideoEncoder* RTCVideoEncoderFactory::CreateVideoEncoder(
- webrtc::VideoCodecType type) {
- for (const auto& codec : codecs_) {
- if (codec.type == type)
- return new RTCVideoEncoder(type, gpu_factories_);
- }
- return nullptr;
-}
-
-const std::vector<cricket::WebRtcVideoEncoderFactory::VideoCodec>&
-RTCVideoEncoderFactory::codecs() const {
- return codecs_;
-}
-
-void RTCVideoEncoderFactory::DestroyVideoEncoder(
- webrtc::VideoEncoder* encoder) {
- delete encoder;
-}
-
-} // namespace content

Powered by Google App Engine
This is Rietveld 408576698