Commit f1b40ad5 authored by Dale Curtis's avatar Dale Curtis Committed by Commit Bot

Reland "Remove constrained baseline profile support for H264 in M86."

This reverts commit a2ecb05c.

Reason for revert: This is causing quality reductions compared to the software encoder. We will need to figure out a way to limit usage of the hardware encoder to higher bit rate situations. See https://crbug.com/1142273

Original change's description:
> Revert "Remove constrained baseline profile support for H264 in M86."
>
> This reverts commit 3234b57f.
>
> Reason for revert: AUD stripping patch from WebRTC has been rolled to Chromium: https://webrtc.googlesource.com/src/+/9345dcfeea73f9dd68dc83dee60255e86dac3f97 and https://source.chromium.org/chromium/chromium/src/+/76165ce5bf99790f8dbae685259723e867458337. CBP for H264 support can come back in.
>
> Original change's description:
> > Remove constrained baseline profile support for H264 in M86.
> >
> > Bug: 982799
> > Change-Id: Ibe20d74bfe6182b8464855106ca2d09f5dca087c
> > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2363462
> > Commit-Queue: Chunbo Hua <chunbo.hua@intel.com>
> > Commit-Queue: Dale Curtis <dalecurtis@chromium.org>
> > Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#799783}
>
> TBR=dalecurtis@chromium.org,chunbo.hua@intel.com
>
> # Not skipping CQ checks because original CL landed > 1 day ago.
>
> Bug: 982799
> Change-Id: Ib1c0212181d7efffe1cd4b82a0a1ddc9a603a0f3
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2427573
> Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
> Commit-Queue: Chunbo Hua <chunbo.hua@intel.com>
> Cr-Commit-Position: refs/heads/master@{#811571}

TBR=dalecurtis@chromium.org,guidou@chromium.org,chunbo.hua@intel.com

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 1142273
Change-Id: I29c06fcba297967bdac8dd188acb24c84c1053bf
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2500446Reviewed-by: default avatarDale Curtis <dalecurtis@chromium.org>
Commit-Queue: Dale Curtis <dalecurtis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#821031}
parent aca5ff00
......@@ -169,8 +169,6 @@ SupportedFormats GetSupportedFormatsInternal(
}
}
cricket::AddH264ConstrainedBaselineProfileToSupportedFormats(
&supported_formats.sdp_formats);
return supported_formats;
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment