• Lambros Lambrou's avatar
    [remoting host] Remove VideoControl lossless-color implementation. · ae177cd5
    Lambros Lambrou authored
    This CL prevents WebRTC clients from changing the VP9 lossless-color
    property via a VideoControl message. This is because WebRTC handles
    that setting via the SDP codec parameter "profile-id=1", which means
    an SDP offer/answer exchange is needed to properly control this
    setting. Non-WebRTC clients are not affected and they can still toggle
    lossless-color via a VideoControl message.
    
    This CL also adds logging to the VP9 encoder so we can verify what
    settings are being applied at the codec level.
    
    This also fixes a bug where the SDP parameter for lossless-color was
    not being respected.
    
    Bug: 1113499
    Change-Id: I15ca83b0fcc157a3b63d3a3ca3509039ed47b24f
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2342126
    Auto-Submit: Lambros Lambrou <lambroslambrou@chromium.org>
    Commit-Queue: Jamie Walch <jamiewalch@chromium.org>
    Reviewed-by: default avatarJamie Walch <jamiewalch@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#795973}
    ae177cd5
webrtc_video_encoder_vpx.cc 23.4 KB