Tracks all open input streams in AudioManagerBase.

media/audio/mac/audio_manager_mac.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "media/audio/mac/audio_manager_mac.h"
 
 #include <algorithm>
 #include <limits>
 #include <vector>
 
@@ skipping 366 matching lines @@
       FROM_HERE, base::Bind(&AudioManagerMac::InitializeOnAudioThread,
                             base::Unretained(this)));
 }
 
 AudioManagerMac::~AudioManagerMac() {
   DCHECK(GetTaskRunner()->BelongsToCurrentThread());
   // We are now in shutdown mode. This flag disables MaybeChangeBufferSize()
   // and IncreaseIOBufferSizeIfPossible() which both touches native Core Audio
   // APIs and they can fail and disrupt tests during shutdown.
   in_shutdown_ = true;
+  Shutdown();

[alokp, 2016/11/15 14:29:38]

Moved here so that proxy streams are closed before the real ones. The proxy ones
may be holding references to real streams.

   // We have seen cases where active input audio is not closed down properly
   // at browser shutdown. AudioInputController::Close() is called but tasks
   // in AudioInputController::DoClose() are not executed. Hence, input streams
   // might remain even at this late state. |low_latency_input_streams_| will be
   // modified during the call to stream->Close(), so we can't iterate over it
   // here.  Instead iterate over a copy.
   // TODO(henrika): figure out the real cause why streams are not closed
   // properly by the AIC for all cases and then remove this loop.
   auto low_latency_input_streams_copy = low_latency_input_streams_;
   for (auto* stream : low_latency_input_streams_copy) {
     LOG(WARNING) << "Closing existing audio input stream at destruction";
     // Prevents active Core Audio callbacks to use possibly invalid objects
     // in its OnData() callback.
     stream->Stop();
     // Avoids hitting CHECK in dtor of AudioManagerBase.
     stream->Close();
   }
   // Perform same type of cleanup as above but for high-latency input streams.
   auto basic_input_streams_copy = basic_input_streams_;
   for (auto* stream : basic_input_streams_copy) {
     LOG(WARNING)
         << "Closing existing high-latency audio input stream at destruction";
     // Prevents active AudioQueue callbacks to use possibly invalid objects
     // in its OnData() callback.
     stream->Stop();
     // Avoids hitting CHECK in dtor of AudioManagerBase.
     stream->Close();
   }
-  Shutdown();
 }
 
 bool AudioManagerMac::HasAudioOutputDevices() {
   return HasAudioHardware(kAudioHardwarePropertyDefaultOutputDevice);
 }
 
 bool AudioManagerMac::HasAudioInputDevices() {
   return HasAudioHardware(kAudioHardwarePropertyDefaultInputDevice);
 }
 
@@ skipping 682 matching lines @@
 ScopedAudioManagerPtr CreateAudioManager(
     scoped_refptr<base::SingleThreadTaskRunner> task_runner,
     scoped_refptr<base::SingleThreadTaskRunner> worker_task_runner,
     AudioLogFactory* audio_log_factory) {
   return ScopedAudioManagerPtr(
       new AudioManagerMac(std::move(task_runner), std::move(worker_task_runner),
                           audio_log_factory));
 }
 
 }  // namespace media