Looking up device id by session id for AudioRendererMixerInput

media/audio/audio_manager_base.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/audio_manager_base.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/command_line.h"
 #include "base/macros.h"
@@ skipping 59 matching lines @@
     //    dispatcher are the same as the request dispatcher.
     return (dispatcher_->input_params.Equals(dispatcher_in->input_params) &&
             dispatcher_->output_params.Equals(dispatcher_in->output_params) &&
             dispatcher_->output_device_id == dispatcher_in->output_device_id);
   }
 
  private:
   const DispatcherParams* dispatcher_;
 };
 
-static bool IsDefaultDeviceId(const std::string& device_id) {
+// static
+bool AudioManagerBase::IsDefaultDeviceId(const std::string& device_id) {

[Guido Urdaneta, 2016/03/08 14:53:09]

Change ARH to use this one instead of the identical function defined there?

[Henrik Grunell, 2016/03/08 21:09:57]

Move this so that the order of member functions is the same as in the header
file.

[o1ka, 2016/04/05 15:13:38]

Done.

[o1ka, 2016/04/05 15:13:38]

Which order is broken?

   return device_id.empty() || device_id == AudioManagerBase::kDefaultDeviceId;
 }
 
 AudioManagerBase::AudioManagerBase(AudioLogFactory* audio_log_factory)
     : max_num_output_streams_(kDefaultMaxOutputStreams),
       max_num_input_streams_(kDefaultMaxInputStreams),
       num_output_streams_(0),
       num_input_streams_(0),
       // TODO(dalecurtis): Switch this to an base::ObserverListThreadSafe, so we
       // don't
@@ skipping 319 matching lines @@
 
   return 0;
 }
 
 scoped_ptr<AudioLog> AudioManagerBase::CreateAudioLog(
     AudioLogFactory::AudioComponent component) {
   return audio_log_factory_->CreateAudioLog(component);
 }
 
 }  // namespace media