Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(158)

Side by Side Diff: webrtc/media/engine/webrtcvideoengine2.cc

Issue 1841373003: Limit max spatial layer to be configured through field trial (3->2) to match current limit in VP9En… (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2014 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2014 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
(...skipping 307 matching lines...) Expand 10 before | Expand all | Expand 10 after
318 bool GetVp9LayersFromFieldTrialGroup(int* num_spatial_layers, 318 bool GetVp9LayersFromFieldTrialGroup(int* num_spatial_layers,
319 int* num_temporal_layers) { 319 int* num_temporal_layers) {
320 std::string group = webrtc::field_trial::FindFullName("WebRTC-SupportVP9SVC"); 320 std::string group = webrtc::field_trial::FindFullName("WebRTC-SupportVP9SVC");
321 if (group.empty()) 321 if (group.empty())
322 return false; 322 return false;
323 323
324 if (sscanf(group.c_str(), "EnabledByFlag_%dSL%dTL", num_spatial_layers, 324 if (sscanf(group.c_str(), "EnabledByFlag_%dSL%dTL", num_spatial_layers,
325 num_temporal_layers) != 2) { 325 num_temporal_layers) != 2) {
326 return false; 326 return false;
327 } 327 }
328 const int kMaxSpatialLayers = 3; 328 const int kMaxSpatialLayers = 2;
329 if (*num_spatial_layers > kMaxSpatialLayers || *num_spatial_layers < 1) 329 if (*num_spatial_layers > kMaxSpatialLayers || *num_spatial_layers < 1)
330 return false; 330 return false;
331 331
332 const int kMaxTemporalLayers = 3; 332 const int kMaxTemporalLayers = 3;
333 if (*num_temporal_layers > kMaxTemporalLayers || *num_temporal_layers < 1) 333 if (*num_temporal_layers > kMaxTemporalLayers || *num_temporal_layers < 1)
334 return false; 334 return false;
335 335
336 return true; 336 return true;
337 } 337 }
338 338
(...skipping 2254 matching lines...) Expand 10 before | Expand all | Expand 10 after
2593 rtx_mapping[video_codecs[i].codec.id] != 2593 rtx_mapping[video_codecs[i].codec.id] !=
2594 fec_settings.red_payload_type) { 2594 fec_settings.red_payload_type) {
2595 video_codecs[i].rtx_payload_type = rtx_mapping[video_codecs[i].codec.id]; 2595 video_codecs[i].rtx_payload_type = rtx_mapping[video_codecs[i].codec.id];
2596 } 2596 }
2597 } 2597 }
2598 2598
2599 return video_codecs; 2599 return video_codecs;
2600 } 2600 }
2601 2601
2602 } // namespace cricket 2602 } // namespace cricket
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698