Skip to content

Commit

Permalink
Address some late-breaking Tricium suggestions from a previous CL
Browse files Browse the repository at this point in the history
Change-Id: I54f2471e25ffc6f3039f4c9fa8225a3a31baa7d2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3869245
Reviewed-by: Lambros Lambrou <lambroslambrou@chromium.org>
Commit-Queue: Joe Downing <joedow@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1042368}
  • Loading branch information
joedow-42 authored and Chromium LUCI CQ committed Sep 1, 2022
1 parent 5bb152d commit 271f647
Showing 1 changed file with 8 additions and 8 deletions.
16 changes: 8 additions & 8 deletions remoting/protocol/webrtc_video_encoder_factory.cc
Original file line number Diff line number Diff line change
Expand Up @@ -22,24 +22,24 @@ namespace remoting::protocol {

WebrtcVideoEncoderFactory::WebrtcVideoEncoderFactory()
: main_task_runner_(base::ThreadTaskRunnerHandle::Get()) {
formats_.emplace_back(webrtc::SdpVideoFormat("VP8"));
formats_.emplace_back(webrtc::SdpVideoFormat("VP9"));
formats_.emplace_back("VP8");
formats_.emplace_back("VP9");
formats_.emplace_back(webrtc::SdpVideoFormat(
"VP9", {{webrtc::kVP9FmtpProfileId,
webrtc::VP9ProfileToString(webrtc::VP9Profile::kProfile1)}}));
formats_.emplace_back("AV1");
formats_.emplace_back(webrtc::SdpVideoFormat(
"AV1",
{{webrtc::kAV1FmtpProfile,
webrtc::AV1ProfileToString(webrtc::AV1Profile::kProfile1).data()}}));
#if defined(USE_H264_ENCODER)
// This call will query the underlying media classes to determine whether
// hardware encoding is supported or not. We use a default resolution and
// framerate so the call doesn't fail due to invalid params.
if (WebrtcVideoEncoderGpu::IsSupportedByH264({{1920, 1080}, 30})) {
formats_.emplace_back(webrtc::SdpVideoFormat("H264"));
formats_.emplace_back("H264");
}
#endif
formats_.emplace_back(webrtc::SdpVideoFormat("AV1"));
formats_.emplace_back(webrtc::SdpVideoFormat(
"AV1",
{{webrtc::kAV1FmtpProfile,
webrtc::AV1ProfileToString(webrtc::AV1Profile::kProfile1).data()}}));
}

WebrtcVideoEncoderFactory::~WebrtcVideoEncoderFactory() = default;
Expand Down

0 comments on commit 271f647

Please sign in to comment.