Revert "Tune vp9 screenshare bitrate and framerate of spatial layers"
This reverts commit aaf3cb3adb618a9f9b14931876b9050201396bee. Reason for revert: Chrome importer consitently failing after this change Original change's description: > Tune vp9 screenshare bitrate and framerate of spatial layers > > VP9 screenshare is not used currently, and with these values according > to local testing with screenshare_loopback, we get performance not worse > than current vp8 settings for similar uplink and downlink values. > > Bug: webrtc:10257 > Change-Id: Icabac04fbd3d616412bbae59291a1fc026d0a504 > Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/126226 > Commit-Queue: Ilya Nikolaevskiy <ilnik@webrtc.org> > Reviewed-by: Johannes Kron <kron@webrtc.org> > Cr-Commit-Position: refs/heads/master@{#27023} TBR=ilnik@webrtc.org,kron@webrtc.org Change-Id: I1ef1eeec8fe87a7662a354ef6362b7d463b2bb4c No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: webrtc:10257 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/126340 Reviewed-by: Jeroen de Borst <jeroendb@webrtc.org> Commit-Queue: Jeroen de Borst <jeroendb@webrtc.org> Cr-Commit-Position: refs/heads/master@{#27027}
This commit is contained in:

committed by
Commit Bot

parent
2c7b9825bc
commit
1109b59d0c
@ -23,10 +23,10 @@ namespace {
|
||||
const size_t kMinVp9SvcBitrateKbps = 30;
|
||||
|
||||
const size_t kMaxNumLayersForScreenSharing = 3;
|
||||
const float kMaxScreenSharingLayerFramerateFps[] = {5.0, 10.0, 30.0};
|
||||
const size_t kMinScreenSharingLayerBitrateKbps[] = {30, 200, 500};
|
||||
const size_t kTargetScreenSharingLayerBitrateKbps[] = {150, 350, 950};
|
||||
const size_t kMaxScreenSharingLayerBitrateKbps[] = {250, 500, 950};
|
||||
const float kMaxScreenSharingLayerFramerateFps[] = {5.0, 5.0, 30.0};
|
||||
const size_t kMinScreenSharingLayerBitrateKbps[] = {30, 150, 500};
|
||||
const size_t kTargetScreenSharingLayerBitrateKbps[] = {150, 350, 1000};
|
||||
const size_t kMaxScreenSharingLayerBitrateKbps[] = {200, 500, 1000};
|
||||
|
||||
} // namespace
|
||||
|
||||
|
@ -54,7 +54,7 @@ TEST(SvcConfig, ScreenSharing) {
|
||||
const SpatialLayer& layer = spatial_layers[i];
|
||||
EXPECT_EQ(layer.width, 1920);
|
||||
EXPECT_EQ(layer.height, 1080);
|
||||
EXPECT_EQ(layer.maxFramerate, (i < 1) ? 5 : (i < 2 ? 10 : 30));
|
||||
EXPECT_EQ(layer.maxFramerate, (i < 2) ? 5 : 30);
|
||||
EXPECT_EQ(layer.numberOfTemporalLayers, 1);
|
||||
EXPECT_LE(layer.minBitrate, layer.maxBitrate);
|
||||
EXPECT_LE(layer.minBitrate, layer.targetBitrate);
|
||||
|
Reference in New Issue
Block a user