Improved error handling in iOS ADM to avoid race during init

webrtc/modules/audio_device/ios/audio_device_ios.h

 /*
  *  Copyright (c) 2012 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 166 matching lines @@
   // a BT headset is enabled or disabled.
   bool RestartAudioUnitWithNewFormat(float sample_rate);
 
   // Activates our audio session, creates and initializes the voice-processing
   // audio unit and verifies that we got the preferred native audio parameters.
   bool InitPlayOrRecord();
 
   // Closes and deletes the voice-processing I/O unit.
   bool ShutdownPlayOrRecord();
 
+  // Helper method for destroying the existing audio unit.
+  void DisposeAudioUnit();
+
   // Callback function called on a real-time priority I/O thread from the audio
   // unit. This method is used to signal that recorded audio is available.
   static OSStatus RecordedDataIsAvailable(
       void* in_ref_con,
       AudioUnitRenderActionFlags* io_action_flags,
       const AudioTimeStamp* time_stamp,
       UInt32 in_bus_number,
       UInt32 in_number_frames,
       AudioBufferList* io_data);
   OSStatus OnRecordedDataIsAvailable(
       AudioUnitRenderActionFlags* io_action_flags,
       const AudioTimeStamp* time_stamp,
       UInt32 in_bus_number,
       UInt32 in_number_frames);
 
   // Callback function called on a real-time priority I/O thread from the audio
   // unit. This method is used to provide audio samples to the audio unit.
   static OSStatus GetPlayoutData(void* in_ref_con,
                                  AudioUnitRenderActionFlags* io_action_flags,
                                  const AudioTimeStamp* time_stamp,
                                  UInt32 in_bus_number,
                                  UInt32 in_number_frames,
                                  AudioBufferList* io_data);
   OSStatus OnGetPlayoutData(AudioUnitRenderActionFlags* io_action_flags,
                             UInt32 in_number_frames,
                             AudioBufferList* io_data);
 
+  static volatile int object_count_;

[tkchin_webrtc, 2015/11/16 21:30:08]

nit: Add comment for what this is used for and why it has to be static

[henrika_webrtc, 2015/11/17 14:46:11]

Done.

+
   // Ensures that methods are called from the same thread as this object is
   // created on.
   rtc::ThreadChecker thread_checker_;
 
   // Raw pointer handle provided to us in AttachAudioBuffer(). Owned by the
   // AudioDeviceModuleImpl class and called by AudioDeviceModuleImpl::Create().
   // The AudioDeviceBuffer is a member of the AudioDeviceModuleImpl instance
   // and therefore outlives this object.
   AudioDeviceBuffer* audio_device_buffer_;
 
@@ skipping 62 matching lines @@
   void* audio_interruption_observer_;
   void* route_change_observer_;
 
   // Contains the audio data format specification for a stream of audio.
   AudioStreamBasicDescription application_format_;
 };
 
 }  // namespace webrtc
 
 #endif  // WEBRTC_MODULES_AUDIO_DEVICE_IOS_AUDIO_DEVICE_IOS_H_