Pass settings model to ARDAppClient instead of individual settings.

webrtc/examples/objc/AppRTCMobile/mac/APPRTCViewController.m

 /*
  *  Copyright 2014 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.
  */
 
@@ skipping 351 matching lines @@
 - (void)appRTCMainView:(APPRTCMainView*)mainView
         didEnterRoomId:(NSString*)roomId
               loopback:(BOOL)isLoopback {
 
   if ([roomId isEqualToString:@""]) {
     [self.mainView displayLogMessage:@"Missing room id"];
     return;
   }
 
   [_client disconnect];
-  ARDAppClient *client = [[ARDAppClient alloc] initWithDelegate:self
-                                               preferVideoCodec:@"H264"];
+  ARDAppClient* client = [[ARDAppClient alloc] initWithDelegate:self];
   [client connectToRoomWithId:roomId
+                usingSettings:[[ARDSettingsModel alloc] init]

[daniela-webrtc, 2017/03/26 15:49:48]

Maybe passing nil here for the settings parameter would be better as it better
conveys the no-op nature of the settings here

[sakal, 2017/03/27 13:34:24]

The reason I do it this way is that I want to use the default values defined by
ARDSettingsModel.

[daniela-webrtc, 2017/03/27 13:49:07]

Acknowledged.

                    isLoopback:isLoopback
                   isAudioOnly:NO
             shouldMakeAecDump:NO
         shouldUseLevelControl:NO];
   _client = client;
 }
 
 #pragma mark - Private
 
 - (APPRTCMainView*)mainView {
@@ skipping 14 matching lines @@
   [self.mainView.remoteVideoView renderFrame:nil];
   [self.mainView.localVideoView renderFrame:nil];
 }
 
 - (void)disconnect {
   [self resetUI];
   [_client disconnect];
 }
 
 @end