Chromium Code Reviews| Index: webrtc/modules/remote_bitrate_estimator/include/bwe_defines.h |
| diff --git a/webrtc/modules/remote_bitrate_estimator/include/bwe_defines.h b/webrtc/modules/remote_bitrate_estimator/include/bwe_defines.h |
| index 3fb7e29e5b2fac5a1232c4bb78fd0cec7c4517f6..9aa82cff976ad1070d5ad3d178c6a9ea7bed59e6 100644 |
| --- a/webrtc/modules/remote_bitrate_estimator/include/bwe_defines.h |
| +++ b/webrtc/modules/remote_bitrate_estimator/include/bwe_defines.h |
| @@ -11,6 +11,7 @@ |
| #ifndef WEBRTC_MODULES_REMOTE_BITRATE_ESTIMATOR_INCLUDE_BWE_DEFINES_H_ |
| #define WEBRTC_MODULES_REMOTE_BITRATE_ESTIMATOR_INCLUDE_BWE_DEFINES_H_ |
| +#include "webrtc/base/optional.h" |
| #include "webrtc/typedefs.h" |
| #define BWE_MAX(a, b) ((a) > (b) ? (a) : (b)) |
| @@ -32,14 +33,14 @@ enum RateControlRegion { kRcNearMax, kRcAboveMax, kRcMaxUnknown }; |
| struct RateControlInput { |
| RateControlInput(BandwidthUsage bw_state, |
| - uint32_t incoming_bitrate, |
| + const rtc::Optional<uint32_t>& incoming_bitrate, |
| double noise_var) |
| : bw_state(bw_state), |
| incoming_bitrate(incoming_bitrate), |
| noise_var(noise_var) {} |
| BandwidthUsage bw_state; |
| - uint32_t incoming_bitrate; |
| + rtc::Optional<uint32_t> incoming_bitrate; |
|
stefan-webrtc
2016/06/02 07:16:41
Seems like a nice improvement!
sprang_webrtc
2016/06/02 08:09:32
Thanks! I think there are bunch of hidden places w
|
| double noise_var; |
| }; |
| } // namespace webrtc |