getUserMedia: handle the device starting status report.

content/browser/renderer_host/media/video_capture_controller.h

Index: content/browser/renderer_host/media/video_capture_controller.h
diff --git a/content/browser/renderer_host/media/video_capture_controller.h b/content/browser/renderer_host/media/video_capture_controller.h
index 634fb169316dadc1adf9aab789e803b493819c11..1923ee8275db99753622f6aafd0a82c27ed61d8a 100644
--- a/content/browser/renderer_host/media/video_capture_controller.h
+++ b/content/browser/renderer_host/media/video_capture_controller.h
@@ -191,6 +191,9 @@ class CONTENT_EXPORT VideoCaptureController : public media::VideoFrameReceiver {
 
   media::VideoCaptureFormat video_capture_format_;
 
+  // True if OnStarted has been reported from the device.
+  bool has_received_onstarted_;

[miu, 2017/03/06 22:37:33]

Instead of adding this separate boolean, how about adding a VideoCaptureState
called VIDEO_CAPTURE_STATE_STARTING? This seems to be an actual concept that is
known and used in both the browser and renderer code.

https://cs.chromium.org/chromium/src/content/common/media/video_capture.h?rcl...

[braveyao, 2017/03/08 22:02:52]

Done.

+
   base::WeakPtrFactory<VideoCaptureController> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(VideoCaptureController);