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

Side by Side Diff: webrtc/modules/remote_bitrate_estimator/bwe_defines.cc

Issue 2986163002: Audit of kConstants missing the const qualifier (Closed)
Patch Set: rebase + constexpr Created 3 years, 4 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
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2016 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2016 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
11 #include "webrtc/modules/remote_bitrate_estimator/include/bwe_defines.h" 11 #include "webrtc/modules/remote_bitrate_estimator/include/bwe_defines.h"
12 #include "webrtc/system_wrappers/include/field_trial.h" 12 #include "webrtc/system_wrappers/include/field_trial.h"
13 13
14 namespace webrtc { 14 namespace webrtc {
15 15
16 const char* kBweTypeHistogram = "WebRTC.BWE.Types"; 16 const char kBweTypeHistogram[] = "WebRTC.BWE.Types";
17 17
18 namespace congestion_controller { 18 namespace congestion_controller {
19 int GetMinBitrateBps() { 19 int GetMinBitrateBps() {
20 constexpr int kAudioMinBitrateBps = 5000; 20 constexpr int kAudioMinBitrateBps = 5000;
21 constexpr int kMinBitrateBps = 10000; 21 constexpr int kMinBitrateBps = 10000;
22 if (webrtc::field_trial::IsEnabled("WebRTC-Audio-SendSideBwe")) { 22 if (webrtc::field_trial::IsEnabled("WebRTC-Audio-SendSideBwe")) {
23 return kAudioMinBitrateBps; 23 return kAudioMinBitrateBps;
24 } 24 }
25 return kMinBitrateBps; 25 return kMinBitrateBps;
26 } 26 }
27 27
28 } // namespace congestion_controller 28 } // namespace congestion_controller
29 29
30 RateControlInput::RateControlInput( 30 RateControlInput::RateControlInput(
31 BandwidthUsage bw_state, 31 BandwidthUsage bw_state,
32 const rtc::Optional<uint32_t>& incoming_bitrate, 32 const rtc::Optional<uint32_t>& incoming_bitrate,
33 double noise_var) 33 double noise_var)
34 : bw_state(bw_state), 34 : bw_state(bw_state),
35 incoming_bitrate(incoming_bitrate), 35 incoming_bitrate(incoming_bitrate),
36 noise_var(noise_var) {} 36 noise_var(noise_var) {}
37 37
38 RateControlInput::~RateControlInput() = default; 38 RateControlInput::~RateControlInput() = default;
39 39
40 } // namespace webrtc 40 } // namespace webrtc
OLDNEW
« no previous file with comments | « webrtc/modules/pacing/alr_detector.cc ('k') | webrtc/modules/remote_bitrate_estimator/include/bwe_defines.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698