Propagate bitrate setting to RTCRtpSender.

webrtc/examples/objc/AppRTCMobile/ios/ARDSettingsStore.m

 /*
  *  Copyright 2016 The WebRTC Project Authors. All rights reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
 #import "ARDSettingsStore.h"
 
 static NSString *const kUserDefaultsMediaConstraintsKey =
     @"rtc_video_resolution_media_constraints_key";
+static NSString *const kUserDefaultsBitrateKey = @"rtc_max_bitrate_key";

[magjed_webrtc, 2016/11/11 08:45:05]

Why do these variable names contain 'UserDefaults'? I.e. wouldn't kBitrateKey be
enough?

[daniela-webrtc, 2016/11/11 13:45:12]

No rule enforcing the UserDefaults in the name. I just like descriptive names
(in true ObjC fashion) :D

[magjed_webrtc, 2016/11/14 13:05:27]

I understand we can name the variable whatever and it will still compile. What I
don't understand is why 'UserDefaults' is good to have in these variable names.
For me this variable looks just like a key and including 'UserDefaults' is
confusing for me since I interpret it as the key can be overridden. I see that
we use these keys in an NSUserDefaults object below and that's where the name
must come from, but for me it's confusing rather than descriptive.

[daniela-webrtc, 2016/11/14 13:36:15]

Done.

 
 NS_ASSUME_NONNULL_BEGIN
 @implementation ARDSettingsStore
 
+- (NSUserDefaults *)storage {
+  return [NSUserDefaults standardUserDefaults];
+}
+
 - (nullable NSString *)videoResolutionConstraintsSetting {
-  return [[NSUserDefaults standardUserDefaults] objectForKey:kUserDefaultsMediaConstraintsKey];
+  return [[self storage] objectForKey:kUserDefaultsMediaConstraintsKey];
 }
 
 - (void)setVideoResolutionConstraintsSetting:(NSString *)constraintsString {
-  [[NSUserDefaults standardUserDefaults] setObject:constraintsString
-                                            forKey:kUserDefaultsMediaConstraintsKey];
+  [[self storage] setObject:constraintsString forKey:kUserDefaultsMediaConstraintsKey];
+}
+
+- (nullable NSNumber *)maxBitrateSetting {
+  return [[self storage] objectForKey:kUserDefaultsBitrateKey];
+}
+
+- (void)setMaxBitrateSetting:(nullable NSNumber *)value {
+  [[self storage] setObject:value forKey:kUserDefaultsBitrateKey];
 }
 
 @end
 NS_ASSUME_NONNULL_END