Mac Video Capture: Connect error logging to WebRTC Log

media/video/capture/mac/video_capture_device_mac.mm

Index: media/video/capture/mac/video_capture_device_mac.mm
diff --git a/media/video/capture/mac/video_capture_device_mac.mm b/media/video/capture/mac/video_capture_device_mac.mm
index fb477190f9a73cf9516d8e92e166d7135585ef4e..9779158a0a017507d462a9f2153cef0f00fda45c 100644
--- a/media/video/capture/mac/video_capture_device_mac.mm
+++ b/media/video/capture/mac/video_capture_device_mac.mm
@@ -378,6 +378,10 @@ void VideoCaptureDeviceMac::AllocateAndStart(
     GetBestMatchSupportedResolution(&width, &height);
 
   client_ = client.Pass();
+  if (device_name_.capture_api_type() == Name::AVFOUNDATION)
+    SendSystemLog("Using AVFoundation for device: " + device_name_.name());

[Henrik Grunell, 2014/05/30 11:48:32]

Just to confirm: device_name_.name() is a user friendly name right? No unique
ID?

[mcasas, 2014/05/30 12:15:27]

Yes, is the camera user friendly name, with the vendor id and product id
suffixed:

"FaceTime HD Camera (Built-in) (058c:8510)"

+  else
+    SendSystemLog("Using QTKit for device: " + device_name_.name());
   NSString* deviceId =
       [NSString stringWithUTF8String:device_name_.id().c_str()];
 
@@ -560,6 +564,12 @@ void VideoCaptureDeviceMac::SetErrorState(const std::string& reason) {
   client_->OnError(reason);
 }
 
+void VideoCaptureDeviceMac::SendSystemLog(const std::string& message) {

[Henrik Grunell, 2014/05/30 11:48:32]

Unclear to me why it's called SendSystemLog. Can you explain or rename.

[mcasas, 2014/05/30 12:15:27]

Renamed to a more simpler name.

+  DCHECK(task_runner_->BelongsToCurrentThread());
+  if (client_)
+    client_->OnLog(message);
+}
+
 bool VideoCaptureDeviceMac::UpdateCaptureResolution() {
  if (![capture_device_ setCaptureHeight:capture_format_.frame_size.height()
                                   width:capture_format_.frame_size.width()