• Henrik Boström's avatar
    [PeerConnection] ThermalResource disabled by default on non-mac. · 1256f171
    Henrik Boström authored
    Prior to this CL, the kWebRtcThermalResource feature was enabled by
    default on all platform, after having been successfully rolled out with
    .gcl. However, only on macOS did the thermal resource actually receive
    any signal from the OS about thermal states, so on non-mac injecting a
    thermal resource into webrtc did not have any observable affects on
    adaptation.
    
    In order to support platform-specific rollout of this feature, this CL
    changes the default to DISABLED on all platforms except the one we are
    confident in: macOS.
    
    We want to do this because in M88, thermal states was implemented on
    ChromeOS, and we want to begin a rollout where the default values of
    the feature is off.
    
    TBR=guidou@chromium.org
    
    Bug: chromium:1144734
    Change-Id: Id1f746013f8a279f5cc4f6b9cd7e48d9a069f70d
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2516940Reviewed-by: default avatarHenrik Boström <hbos@chromium.org>
    Reviewed-by: default avatarGuido Urdaneta <guidou@chromium.org>
    Commit-Queue: Henrik Boström <hbos@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#823541}
    1256f171
thermal_resource.cc 3.03 KB