Switch event type when a capture device has been stopped from the render process. This make sure th…

content/renderer/media/rtc_video_capturer.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/renderer/media/rtc_video_capturer.h"
 
 #include "base/bind.h"
 #include "base/debug/trace_event.h"
 
 namespace content {
@@ skipping 116 matching lines @@
                        frame.time_stamp);
 
   // This signals to libJingle that a new VideoFrame is available.
   // libJingle have no assumptions on what thread this signal come from.
   SignalFrameCaptured(this, &frame);
 }
 
 void RtcVideoCapturer::OnStateChange(
     RtcVideoCaptureDelegate::CaptureState state) {
   cricket::CaptureState converted_state = cricket::CS_FAILED;
+  DVLOG(3) << " RtcVideoCapturer::OnStateChange " << state;
   switch (state) {
     case RtcVideoCaptureDelegate::CAPTURE_STOPPED:
       converted_state = cricket::CS_STOPPED;
       break;
     case RtcVideoCaptureDelegate::CAPTURE_RUNNING:
       converted_state = cricket::CS_RUNNING;
       break;
     case RtcVideoCaptureDelegate::CAPTURE_FAILED:
+      // TODO(perkj): Update the comments in the the definition of
+      // cricket::CS_FAILED. According to the comments, cricket::CS_FAILED
+      // means that the capturer failed to start. But here and in libjingle it
+      // is also used if an error occur during capturing.
       converted_state = cricket::CS_FAILED;
       break;
     default:
       NOTREACHED();
       break;
   }
   SignalStateChange(this, converted_state);
 }
 
 }  // namespace content