Reland "Reland "media: use VEA SupportedProfile |min_resolution| for PeerConnection""
This reverts commit 2f4fb261dc73ab21eb044766706eaeb87286a99f.
Reason for revert: M80/M81 blocker. Bug 1046501.
Original change's description:
> Revert "Reland "media: use VEA SupportedProfile |min_resolution| for PeerConnection""
>
> This reverts commit 49e19ab94fbc8cd967c892ff88755b866ba01d9a.
>
> Reason for revert: Landed without testing (No-Try: true). Will revert, enable tests and reland.
>
> Original change's description:
> > Reland "media: use VEA SupportedProfile |min_resolution| for PeerConnection"
> >
> > This reverts commit a629c52fa61d6885b51bf145e265ad49bf4c3e0c.
> >
> > Reason for revert: M80/81 blocker. Bug 1046501.
> >
> > Original change's description:
> > > Revert "media: use VEA SupportedProfile |min_resolution| for PeerConnection"
> > >
> > > This reverts commit 72037f8203ffc84e0833a0c56d49e98654e2caf9.
> > >
> > > Reason for revert: M80 Stable blocker bug 1046501
> > >
> > > Original change's description:
> > > > media: use VEA SupportedProfile |min_resolution| for PeerConnection
> > > >
> > > > crrev.com/c/1828078 added and wired the video encode acceleration
> > > > minimum resolution; this CL uses this information from peerconnection's
> > > > rtc_video_encoder.cc and video_codec_factory.cc to avoid using a
> > > > hw accelerator for resolutions below the given one.
> > > >
> > > > TEST=https://codepen.io/miguelao/full/jONQBYb on nocturne (pixel slate), then
> > > > inspecting chrome:webrtc-internals, verifying that resolutions below 321x241
> > > > are encoded using libvpx while larger ones are encoded using "ExternalEncoder",
> > > > i.e. the video encode accelerator in Chrome, in this case VA-API.
> > > >
> > > > Bug: 1008491
> > > > Change-Id: Iccfe6be8811ca265d6dbc742beb3ec0901406a40
> > > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1848398
> > > > Commit-Queue: Miguel Casas <[email protected]>
> > > > Reviewed-by: Guido Urdaneta <[email protected]>
> > > > Reviewed-by: Kentaro Hara <[email protected]>
> > > > Cr-Commit-Position: refs/heads/master@{#720981}
> > >
> > > [email protected],[email protected],[email protected],[email protected],[email protected],[email protected]
> > >
> > > # Not skipping CQ checks because original CL landed > 1 day ago.
> > >
> > > Bug: 1008491
> > > Change-Id: Ie7279fcc02e041f4e1a4a0ed0cef248aa24d472e
> > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2028889
> > > Reviewed-by: Miguel Casas <[email protected]>
> > > Reviewed-by: Guido Urdaneta <[email protected]>
> > > Reviewed-by: Henrik Boström <[email protected]>
> > > Commit-Queue: Miguel Casas <[email protected]>
> > > Cr-Commit-Position: refs/heads/master@{#736904}
> >
> > [email protected],[email protected],[email protected],[email protected],[email protected],[email protected],[email protected],[email protected]
> >
> > Change-Id: I92b4e10f9ef39d2c642a6c9ae4d3a1e3f0fbb609
> > No-Presubmit: true
> > No-Tree-Checks: true
> > No-Try: true
> > Bug: 1008491
> > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2033345
> > Commit-Queue: Miguel Casas <[email protected]>
> > Reviewed-by: Miguel Casas <[email protected]>
> > Cr-Commit-Position: refs/heads/master@{#737332}
>
> [email protected],[email protected],[email protected],[email protected],[email protected],[email protected],[email protected],[email protected],[email protected]
>
> Change-Id: I1cadec33578747fca04b44867a8951a9c69d3280
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Bug: 1008491
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2033348
> Commit-Queue: Ben Mason <[email protected]>
> Reviewed-by: Ben Mason <[email protected]>
> Cr-Commit-Position: refs/heads/master@{#737334}
[email protected],[email protected],[email protected],[email protected],[email protected],[email protected],[email protected],[email protected],[email protected],[email protected]
Bug: 1008491
Change-Id: I4d44659b5bd9fb85cd3f1121a5633fb1b183424d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2033257
Reviewed-by: Ben Mason <[email protected]>
Commit-Queue: Sergey Silkin <[email protected]>
Cr-Commit-Position: refs/heads/master@{#737361}
diff --git a/chrome/browser/flag_descriptions.h b/chrome/browser/flag_descriptions.h
index 52ae1c3..6df1bd62 100644
--- a/chrome/browser/flag_descriptions.h
+++ b/chrome/browser/flag_descriptions.h
@@ -1231,6 +1231,9 @@
extern const char kWebrtcStunOriginName[];
extern const char kWebrtcStunOriginDescription[];
+extern const char kWebrtcUseMinMaxVEADimensionsName[];
+extern const char kWebrtcUseMinMaxVEADimensionsDescription[];
+
#if BUILDFLAG(ENABLE_WEBUI_TAB_STRIP)
extern const char kWebUITabStripName[];
extern const char kWebUITabStripDescription[];
