Implement browser-side logging to WebRtc log

content/browser/renderer_host/media/audio_input_device_manager.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 "content/browser/renderer_host/media/audio_input_device_manager.h"
 
 #include "base/bind.h"
 #include "base/memory/scoped_ptr.h"
+#include "content/browser/renderer_host/media/media_stream_manager.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/common/media_stream_request.h"
 #include "media/audio/audio_device_name.h"
 #include "media/audio/audio_input_ipc.h"
 #include "media/audio/audio_manager_base.h"
 #include "media/audio/audio_parameters.h"
 #include "media/base/channel_layout.h"
 #include "media/base/scoped_histogram_timer.h"
 
 namespace content {
@@ skipping 112 matching lines @@
       // AudioManager is guaranteed to outlive MediaStreamManager in
       // BrowserMainloop.
       audio_manager_->GetAudioInputDeviceNames(&device_names);
       break;
 
     default:
       NOTREACHED();
       break;
   }
 
+  MediaStreamManager::AddLogMessage(

[perkj_chrome, 2014/01/20 11:07:08]

Why do we have to log here? 
Why not do it in MediaStreamManager::DevicesEnumerated? 

[vrk (LEFT CHROMIUM), 2014/01/22 02:25:42]

Sure, I can do that. I originally put more logging here but then moved it out of
this CL. Agree it makes more sense on MSM::DevicesEnumerated.

That said, I wanted to put an example of MediaStreamManager::AddLogMessage in
this CL. Device enumeration is not the only thing we want to log, but I was
going to put other log messages in separate CLs because they span all OSes
(didn't want to make the CL too big). So I added to this CL log messages on
video device error. PTAL.

+      "AIDM::EODTT found the following devices:");

[Henrik Grunell, 2014/01/17 08:51:25]

Same comments as in VideoCaptureManager for this and the next change.

[vrk (LEFT CHROMIUM), 2014/01/22 02:25:42]

Done.

   scoped_ptr<StreamDeviceInfoArray> devices(new StreamDeviceInfoArray());
   for (media::AudioDeviceNames::iterator it = device_names.begin();
        it != device_names.end(); ++it) {
     // Add device information to device vector.
     devices->push_back(StreamDeviceInfo(
         stream_type, it->device_name, it->unique_id));
+    MediaStreamManager::AddLogMessage(it->device_name + "(" + it->unique_id);
   }
 
   // If the |use_fake_device_| flag is on, inject the fake device if there is
   // no available device on the OS.
   if (use_fake_device_ && devices->empty()) {
     devices->push_back(StreamDeviceInfo(
         stream_type, media::AudioManagerBase::kDefaultDeviceName,
         media::AudioManagerBase::kDefaultDeviceId));
   }
 
@@ skipping 92 matching lines @@
   for (StreamDeviceList::iterator i(devices_.begin()); i != devices_.end();
        ++i) {
     if (i->session_id == session_id)
       return i;
   }
 
   return devices_.end();
 }
 
 }  // namespace content