• Sergey Silkin's avatar
    Reland "media: use VEA SupportedProfile |min_resolution| for PeerConnection" · 49e19ab9
    Sergey Silkin authored
    This reverts commit a629c52f.
    
    Reason for revert: M80/81 blocker. Bug 1046501.
    
    Original change's description:
    > Revert "media: use VEA SupportedProfile |min_resolution| for PeerConnection"
    > 
    > This reverts commit 72037f82.
    > 
    > 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 <mcasas@chromium.org>
    > > Reviewed-by: Guido Urdaneta <guidou@chromium.org>
    > > Reviewed-by: Kentaro Hara <haraken@chromium.org>
    > > Cr-Commit-Position: refs/heads/master@{#720981}
    > 
    > TBR=mcasas@chromium.org,haraken@chromium.org,sprang@chromium.org,hbos@google.com,guidou@chromium.org,ilnik@chromium.org
    > 
    > # 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 <mcasas@chromium.org>
    > Reviewed-by: Guido Urdaneta <guidou@chromium.org>
    > Reviewed-by: Henrik Boström <hbos@chromium.org>
    > Commit-Queue: Miguel Casas <mcasas@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#736904}
    
    TBR=mcasas@chromium.org,haraken@chromium.org,sprang@chromium.org,hbos@google.com,hbos@chromium.org,guidou@chromium.org,ilnik@chromium.org,ssilkin@chromium.org
    
    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 <mcasas@chromium.org>
    Reviewed-by: default avatarMiguel Casas <mcasas@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#737332}
    49e19ab9
enums.xml 2.96 MB
The source could not be displayed because it is larger than 1 MB. You can load it anyway or download it instead.